Bug 193849 – Add a scrolling tree test that toggles scrollability on an intermediate iframe

• Status: RESOLVED
• Resolution: FIXED
• Component: New Bugs

[Simon Fraser (smfr)]

Add a scrolling tree test that toggles scrollability on an intermediate iframe

[Simon Fraser (smfr)]

Created attachment 360174 [details]
Patch

[Dean Jackson]

Comment on attachment 360174 [details]
Patch

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

> LayoutTests/scrollingcoordinator/scrolling-tree/resources/scroll-toggling-frame.html:45
> +            iframe {
> +                width: 400px;
> +                height: 300px;
> +                box-shadow: 0 0 10px black;
> +                border: 2px solid black;
> +            }

I don't see any iframe in this doc.

[Simon Fraser (smfr)]

https://trac.webkit.org/changeset/240535/webkit

[Radar WebKit Bug Importer]

<rdar://problem/47575885>

[Truitt Savell]

This test scrollingcoordinator/scrolling-tree/toggle-coordinated-frame-scrolling.html

that was added is a flakey failure. 

History:
https://webkit-test-results.webkit.org/dashboards/flakiness_dashboard.html#showAllRuns=true&tests=scrollingcoordinator%2Fscrolling-tree%2Ftoggle-coordinated-frame-scrolling.html

Diff:
--- /Volumes/Data/slave/highsierra-debug-tests-wk2/build/layout-test-results/scrollingcoordinator/scrolling-tree/toggle-coordinated-frame-scrolling-expected.txt
+++ /Volumes/Data/slave/highsierra-debug-tests-wk2/build/layout-test-results/scrollingcoordinator/scrolling-tree/toggle-coordinated-frame-scrolling-actual.txt
@@ -33,27 +33,6 @@
           (min layout viewport origin (0,0))
           (max layout viewport origin (0,524))
           (behavior for fixed 0)
-          (children 1
-            (Frame hosting node
-              (children 1
-                (Frame scrolling node
-                  (scrollable area size 485 400)
-                  (contents size 485 1024)
-                  (scrollable area parameters 
-                    (horizontal scroll elasticity 0)
-                    (vertical scroll elasticity 0)
-                    (horizontal scrollbar mode 0)
-                    (vertical scrollbar mode 0)
-                    (has enabled vertical scrollbar 1))
-                  (visual viewport enabled 1)
-                  (layout viewport at (0,0) size 485x400)
-                  (min layout viewport origin (0,0))
-                  (max layout viewport origin (0,624))
-                  (behavior for fixed 0)
-                )
-              )
-            )
-          )
         )
       )
     )
@@ -65,7 +44,7 @@
 
 (Frame scrolling node
   (scrollable area size 785 600)
-  (contents size 785 1541)
+  (contents size 785 1226)
   (scrollable area parameters 
     (horizontal scroll elasticity 2)
     (vertical scroll elasticity 2)
@@ -75,7 +54,7 @@
   (visual viewport enabled 1)
   (layout viewport at (0,0) size 785x600)
   (min layout viewport origin (0,0))
-  (max layout viewport origin (0,941))
+  (max layout viewport origin (0,626))
   (behavior for fixed 0)
   (children 1
     (Frame hosting node

[Truitt Savell]

Made bug https://bugs.webkit.org/show_bug.cgi?id=194291 to track this test failure