RESOLVED FIXED164965
Convert IntersectionObserver over to using RuntimeEnabledFeatures so it can be properly excluded from script 
https://bugs.webkit.org/show_bug.cgi?id=164965
Summary Convert IntersectionObserver over to using RuntimeEnabledFeatures so it can b...
Sam Weinig
Reported 2016-11-18 15:48:37 PST
Convert IntersectionObserver over to using RuntimeEnabledFeatures so it can be properly excluded from script
Attachments
Patch (31.45 KB, patch)
2016-11-18 16:32 PST, Sam Weinig 		no flags
DetailsFormatted DiffDiff
Patch (31.45 KB, patch)
2016-11-18 16:47 PST, Sam Weinig 		no flags
DetailsFormatted DiffDiff
Patch (33.17 KB, patch)
2016-11-26 15:39 PST, Simon Fraser (smfr) 		no flags
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Sam Weinig
Comment 1 2016-11-18 16:32:18 PST
Created attachment 295216 [details] Patch
Sam Weinig
Comment 2 2016-11-18 16:47:56 PST
Created attachment 295222 [details] Patch
Simon Fraser (smfr)
Comment 3 2016-11-26 15:39:26 PST
Created attachment 295443 [details] Patch
WebKit Commit Bot
Comment 4 2016-11-26 15:40:45 PST
Attachment 295443 [details] did not pass style-queue: ERROR: Tools/DumpRenderTree/TestOptions.mm:29: Streams are highly discouraged. [readability/streams] [3] Total errors found: 1 in 27 files If any of these errors are false positives, please file a bug against check-webkit-style.
WebKit Commit Bot
Comment 5 2016-11-26 20:22:06 PST
Comment on attachment 295443 [details] Patch Clearing flags on attachment: 295443 Committed r208983: <http://trac.webkit.org/changeset/208983>
WebKit Commit Bot
Comment 6 2016-11-26 20:22:11 PST
All reviewed patches have been landed. Closing bug.
Csaba Osztrogonác
Comment 7 2016-11-26 23:25:55 PST
(In reply to comment #5) > Comment on attachment 295443 [details] > Patch > > Clearing flags on attachment: 295443 > > Committed r208983: <http://trac.webkit.org/changeset/208983> It broke the Apple Mac cmake build, see build.webkit.org for details.
Simon Fraser (smfr)
Comment 8 2016-11-26 23:34:27 PST
Attempted a fix in https://trac.webkit.org/changeset/208987
Ryan Haddad
Comment 9 2016-11-27 20:16:05 PST
(In reply to comment #5) > Comment on attachment 295443 [details] > Patch > > Clearing flags on attachment: 295443 > > Committed r208983: <http://trac.webkit.org/changeset/208983> The LayoutTest edited with this change is failing on macOS and iOS WK2: https://webkit-test-results.webkit.org/dashboards/flakiness_dashboard.html#showAllRuns=true&tests=intersection-observer%2Fintersection-observer-entry-interface.html
Simon Fraser (smfr)
Comment 10 2016-11-27 23:20:03 PST
I'll take a look.
Simon Fraser (smfr)
Comment 11 2016-11-28 16:36:15 PST
I can't reproduce that failure locally.
Simon Fraser (smfr)
Comment 12 2016-11-28 22:47:07 PST
I filed bug 165133 to fix that.
Note You need to log in before you can comment on or make changes to this bug.