Bug 195295 – [ContentChangeObserver] Introduce fixed duration content observation

• Status: RESOLVED
• Resolution: FIXED
• Component: Layout and Rendering

[zalan]

32ms for now.

[zalan]

Created attachment 363554 [details]
Patch

[Radar WebKit Bug Importer]

<rdar://problem/48579913>

[zalan]

Created attachment 363557 [details]
Patch

[Simon Fraser (smfr)]

Comment on attachment 363557 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=363557&action=review

> Source/WebCore/ChangeLog:10
> +        It helps finding content changes when they are triggred by nested timers (using a 32ms window). Currently nested timers get dropped on the floor and
> +        we stop observing for content changes before they get fired. 

Maybe some words about what web pages are trying to do.

> Source/WebCore/page/ios/ContentChangeObserver.h:132
> +        InstalledFixedObservationTimeWindow,

Started?

> Source/WebCore/page/ios/ContentChangeObserver.h:133
> +        RemovedFixedObservationTimeWindow

Ended

[zalan]

Created attachment 363565 [details]
Patch

[WebKit Commit Bot]

Comment on attachment 363565 [details]
Patch

Clearing flags on attachment: 363565

Committed r242403: <https://trac.webkit.org/changeset/242403>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Ryan Haddad]

This change caused multiple layout test crashes on iOS Simulator:

Thread 0 Crashed:: Dispatch queue: com.apple.main-thread
0   com.apple.WebCore             	0x000000010a937be7 WebCore::ContentChangeObserver::stopDurationBasedContentObservation() + 87 (ContentChangeObserver.cpp:236)
1   com.apple.WebCore             	0x000000010a978b3d WebCore::ThreadTimers::sharedTimerFiredInternal() + 173 (ThreadTimers.cpp:132)
2   com.apple.WebCore             	0x000000010a99e4ff WebCore::timerFired(__CFRunLoopTimer*, void*) + 31 (MainThreadSharedTimerCF.cpp:75)
3   com.apple.CoreFoundation      	0x0000000103e55f34 __CFRUNLOOP_IS_CALLING_OUT_TO_A_TIMER_CALLBACK_FUNCTION__ + 20
4   com.apple.CoreFoundation      	0x0000000103e55b32 __CFRunLoopDoTimer + 1026
5   com.apple.CoreFoundation      	0x0000000103e5539a __CFRunLoopDoTimers + 266
6   com.apple.CoreFoundation      	0x0000000103e4fa1c __CFRunLoopRun + 2252
7   com.apple.CoreFoundation      	0x0000000103e4ee11 CFRunLoopRunSpecific + 625
8   com.apple.Foundation          	0x00000001018b6322 -[NSRunLoop(NSRunLoop) runMode:beforeDate:] + 277
9   com.apple.Foundation          	0x00000001018b6492 -[NSRunLoop(NSRunLoop) run] + 76
10  libxpc.dylib                  	0x0000000105649812 _xpc_objc_main + 460
11  libxpc.dylib                  	0x000000010564bcbd xpc_main + 143
12  com.apple.WebKit              	0x0000000101f34a40 WebKit::XPCServiceMain(int, char const**) + 403 (OSObjectPtr.h:69)
13  libdyld.dylib                 	0x0000000105335575 start + 1

https://build.webkit.org/results/Apple%20iOS%2012%20Simulator%20Release%20WK2%20(Tests)/r242464%20(2967)/results.html

[Ryan Haddad]

Reverted r242403 for reason:

Caused layout test crashes on iOS simulator.

Committed r242467: <https://trac.webkit.org/changeset/242467>

[Truitt Savell]

*** Bug 195334 has been marked as a duplicate of this bug. ***

[Shawn Roberts]

Was causing several layout test failures as well :

fast/shadow-dom/click-on-slotted-anchor-with-hover.html
fast/images/imagemap-in-shadow-tree.html
fast/events/click-handler-on-body-simple.html
fast/forms/datalist/datalist-textinput-suggestions-order.html
fast/events/beforeunload-alert-user-interaction2.html
http/tests/adClickAttribution/anchor-tag-attributes-validation.html
fast/scrolling/ios/hit-testing-iframe-001.html
fast/scrolling/ios/hit-testing-iframe-002.html
fast/scrolling/ios/hit-testing-iframe-003.html
fast/scrolling/ios/hit-testing-iframe-004.html
fast/scrolling/ios/hit-testing-iframe-005.html
fast/scrolling/ios/mixing-user-and-programmatic-scroll-006.html

Dashboard :

https://webkit-test-results.webkit.org/dashboards/flakiness_dashboard.html#showAllRuns=true&tests=fast%2Fshadow-dom%2Fclick-on-slotted-anchor-with-hover.html%20fast%2Fimages%2Fimagemap-in-shadow-tree.html%20fast%2Fevents%2Fclick-handler-on-body-simple.html%20fast%2Fforms%2Fdatalist%2Fdatalist-textinput-suggestions-order.html%20fast%2Fevents%2Fbeforeunload-alert-user-interaction2.html%20http%2Ftests%2FadClickAttribution%2Fanchor-tag-attributes-validation.html%20fast%2Fscrolling%2Fios%2Fhit-testing-iframe-001.html%20fast%2Fscrolling%2Fios%2Fhit-testing-iframe-002.html%20fast%2Fscrolling%2Fios%2Fhit-testing-iframe-003.html%20fast%2Fscrolling%2Fios%2Fhit-testing-iframe-004.html%20fast%2Fscrolling%2Fios%2Fhit-testing-iframe-005.html%20fast%2Fscrolling%2Fios%2Fmixing-user-and-programmatic-scroll-006.html

[zalan]

Created attachment 363884 [details]
Patch

[zalan]

Created attachment 363966 [details]
Patch

[WebKit Commit Bot]

Comment on attachment 363966 [details]
Patch

Clearing flags on attachment: 363966

Committed r242628: <https://trac.webkit.org/changeset/242628>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.