Bug 172825

Summary: [Mac] Include frames in the scrolling tree when ScrollingTreeIncludesFrames=true
Product: WebKit Reporter: Frédéric Wang (:fredw) <fred.wang>
Component: FramesAssignee: Frédéric Wang (:fredw) <fred.wang>
Status: RESOLVED FIXED    
Severity: Normal CC: buildbot, fred.wang, rniwa, simon.fraser
Priority: P2    
Version: Other   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 172851    
Bug Blocks: 171667    
Attachments:
Description Flags
Patch (WIP)
buildbot: commit-queue-
Archive of layout-test-results from ews123 for ios-simulator-wk2
none
Patch (172851 + 172825)
buildbot: commit-queue-
Archive of layout-test-results from ews102 for mac-elcapitan
none
Archive of layout-test-results from ews116 for mac-elcapitan
none
Archive of layout-test-results from ews124 for ios-simulator-wk2
none
Patch
simon.fraser: review+, buildbot: commit-queue-
Archive of layout-test-results from ews104 for mac-elcapitan-wk2
none
Patch none

Description Frédéric Wang (:fredw) 2017-06-01 11:07:58 PDT
Attempt to extract this from bug 171667
Comment 1 Frédéric Wang (:fredw) 2017-06-01 11:08:29 PDT
Created attachment 311729 [details]
Patch (WIP)
Comment 2 Build Bot 2017-06-01 12:53:47 PDT
Comment on attachment 311729 [details]
Patch (WIP)

Attachment 311729 [details] did not pass ios-sim-ews (ios-simulator-wk2):
Output: http://webkit-queues.webkit.org/results/3855104

New failing tests:
compositing/iframes/scrolling-iframe.html
Comment 3 Build Bot 2017-06-01 12:53:48 PDT
Created attachment 311742 [details]
Archive of layout-test-results from ews123 for ios-simulator-wk2

The attached test failures were seen while running run-webkit-tests on the ios-sim-ews.
Bot: ews123  Port: ios-simulator-wk2  Platform: Mac OS X 10.12.5
Comment 4 Frédéric Wang (:fredw) 2017-06-02 06:15:23 PDT
Created attachment 311819 [details]
Patch (172851 + 172825)
Comment 5 Build Bot 2017-06-02 07:13:18 PDT
Comment on attachment 311819 [details]
Patch (172851 + 172825)

Attachment 311819 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.webkit.org/results/3859695

New failing tests:
fast/scrolling/scrolling-tree-includes-frame.html
Comment 6 Build Bot 2017-06-02 07:13:19 PDT
Created attachment 311820 [details]
Archive of layout-test-results from ews102 for mac-elcapitan

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews102  Port: mac-elcapitan  Platform: Mac OS X 10.11.6
Comment 7 Build Bot 2017-06-02 07:34:50 PDT
Comment on attachment 311819 [details]
Patch (172851 + 172825)

Attachment 311819 [details] did not pass mac-debug-ews (mac):
Output: http://webkit-queues.webkit.org/results/3859713

New failing tests:
fast/scrolling/scrolling-tree-includes-frame.html
Comment 8 Build Bot 2017-06-02 07:34:52 PDT
Created attachment 311821 [details]
Archive of layout-test-results from ews116 for mac-elcapitan

The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews116  Port: mac-elcapitan  Platform: Mac OS X 10.11.6
Comment 9 Build Bot 2017-06-02 07:44:48 PDT
Comment on attachment 311819 [details]
Patch (172851 + 172825)

Attachment 311819 [details] did not pass ios-sim-ews (ios-simulator-wk2):
Output: http://webkit-queues.webkit.org/results/3859722

New failing tests:
compositing/iframes/scrolling-iframe.html
Comment 10 Build Bot 2017-06-02 07:44:50 PDT
Created attachment 311824 [details]
Archive of layout-test-results from ews124 for ios-simulator-wk2

The attached test failures were seen while running run-webkit-tests on the ios-sim-ews.
Bot: ews124  Port: ios-simulator-wk2  Platform: Mac OS X 10.12.5
Comment 11 Frédéric Wang (:fredw) 2017-06-02 07:57:59 PDT
Created attachment 311825 [details]
Patch

Simon: I'm not yet 100% sure of this on all these platforms and the patch applies on bug 172851 anyway, but I'm asking review to get some feedback.
Comment 12 Build Bot 2017-06-02 09:02:08 PDT
Comment on attachment 311825 [details]
Patch

Attachment 311825 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.webkit.org/results/3860071

New failing tests:
fast/scrolling/scrolling-tree-includes-frame.html
Comment 13 Build Bot 2017-06-02 09:02:10 PDT
Created attachment 311829 [details]
Archive of layout-test-results from ews104 for mac-elcapitan-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews104  Port: mac-elcapitan-wk2  Platform: Mac OS X 10.11.6
Comment 14 Frédéric Wang (:fredw) 2017-06-02 11:50:10 PDT
Created attachment 311842 [details]
Patch

Same patch, but try again after r217726.
Comment 15 Frédéric Wang (:fredw) 2017-06-02 12:40:14 PDT
Committed r217730: <http://trac.webkit.org/changeset/217730>