RESOLVED FIXED 172851
[Mac] Use compositing for frames when ScrollingTreeIncludesFrames=true
https://bugs.webkit.org/show_bug.cgi?id=172851
Summary [Mac] Use compositing for frames when ScrollingTreeIncludesFrames=true
Frédéric Wang (:fredw)
Reported 2017-06-02 01:59:57 PDT
This piece is extracted from bug 171667, where we will need a separate layer for the scrollable frames. For now, we will only do that when ScrollingTreeIncludesFrames=true, so that we do not change the default behavior.
Attachments
Patch (8.37 KB, patch)
2017-06-02 02:18 PDT, Frédéric Wang (:fredw)
no flags
Archive of layout-test-results from ews103 for mac-elcapitan (998.26 KB, application/zip)
2017-06-02 03:17 PDT, Build Bot
no flags
Archive of layout-test-results from ews113 for mac-elcapitan (1.74 MB, application/zip)
2017-06-02 03:39 PDT, Build Bot
no flags
Patch (8.72 KB, patch)
2017-06-02 03:48 PDT, Frédéric Wang (:fredw)
simon.fraser: review+
Frédéric Wang (:fredw)
Comment 1 2017-06-02 02:18:12 PDT
Build Bot
Comment 2 2017-06-02 03:17:11 PDT
Comment on attachment 311809 [details] Patch Attachment 311809 [details] did not pass mac-ews (mac): Output: http://webkit-queues.webkit.org/results/3858870 New failing tests: compositing/iframes/compositing-for-scrollable-iframe.html
Build Bot
Comment 3 2017-06-02 03:17:12 PDT
Created attachment 311813 [details] Archive of layout-test-results from ews103 for mac-elcapitan The attached test failures were seen while running run-webkit-tests on the mac-ews. Bot: ews103 Port: mac-elcapitan Platform: Mac OS X 10.11.6
Build Bot
Comment 4 2017-06-02 03:38:59 PDT
Comment on attachment 311809 [details] Patch Attachment 311809 [details] did not pass mac-debug-ews (mac): Output: http://webkit-queues.webkit.org/results/3858895 New failing tests: compositing/iframes/compositing-for-scrollable-iframe.html
Build Bot
Comment 5 2017-06-02 03:39:00 PDT
Created attachment 311816 [details] Archive of layout-test-results from ews113 for mac-elcapitan The attached test failures were seen while running run-webkit-tests on the mac-debug-ews. Bot: ews113 Port: mac-elcapitan Platform: Mac OS X 10.11.6
Frédéric Wang (:fredw)
Comment 6 2017-06-02 03:48:12 PDT
Simon Fraser (smfr)
Comment 7 2017-06-02 10:58:49 PDT
Comment on attachment 311817 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=311817&action=review > Source/WebKit2/WebProcess/WebCoreSupport/WebChromeClient.h:229 > + ScrollableInnerFrameTrigger | Maybe ScrollableNonMainFrameTrigger? I don't actually know if anyone uses these triggers any more, now that Blink is out of the repo.
Frédéric Wang (:fredw)
Comment 8 2017-06-02 11:42:14 PDT
Frédéric Wang (:fredw)
Comment 9 2017-06-02 11:43:45 PDT
(In reply to Simon Fraser (smfr) from comment #7) > I don't actually know if anyone uses these triggers any more, now that Blink > is out of the repo. Yes, this is actually only used in RenderLayerCompositor::requiresCompositingForScrollableFrame ; I renamed the flag as you suggest.
Note You need to log in before you can comment on or make changes to this bug.