scrollingcoordinator/ios/ui-scrolling-tree.html is a Flaky Failure on iPad
authorsimon.fraser@apple.com <simon.fraser@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 21 Oct 2019 20:00:24 +0000 (20:00 +0000)
committersimon.fraser@apple.com <simon.fraser@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 21 Oct 2019 20:00:24 +0000 (20:00 +0000)
https://bugs.webkit.org/show_bug.cgi?id=203119
rdar://problem/52970947

Reviewed by Wenson Hsieh.

iPad viewport heuristics (WebPage::immediatelyShrinkToFitContent()) run on a zero-delay
timer after page load, which races with this test getting the UI-side scrolling tree
and makes the test flakey.

Fix by turning off the heuristics with "contentMode=mobile". Also fix the test
to use UIHelper to get the scrolling tree.

* platform/ipad/TestExpectations:
* platform/ipad/scrollingcoordinator/ios/ui-scrolling-tree-expected.txt:
* scrollingcoordinator/ios/ui-scrolling-tree.html:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@251375 268f45cc-cd09-0410-ab3c-d52691b4dbfc

LayoutTests/ChangeLog
LayoutTests/platform/ipad/TestExpectations
LayoutTests/platform/ipad/scrollingcoordinator/ios/ui-scrolling-tree-expected.txt
LayoutTests/scrollingcoordinator/ios/ui-scrolling-tree.html

index 45f3a44..905c06c 100644 (file)
@@ -1,5 +1,24 @@
 2019-10-21  Simon Fraser  <simon.fraser@apple.com>
 
+        scrollingcoordinator/ios/ui-scrolling-tree.html is a Flaky Failure on iPad
+        https://bugs.webkit.org/show_bug.cgi?id=203119
+        rdar://problem/52970947
+
+        Reviewed by Wenson Hsieh.
+
+        iPad viewport heuristics (WebPage::immediatelyShrinkToFitContent()) run on a zero-delay
+        timer after page load, which races with this test getting the UI-side scrolling tree
+        and makes the test flakey.
+
+        Fix by turning off the heuristics with "contentMode=mobile". Also fix the test
+        to use UIHelper to get the scrolling tree.
+
+        * platform/ipad/TestExpectations:
+        * platform/ipad/scrollingcoordinator/ios/ui-scrolling-tree-expected.txt:
+        * scrollingcoordinator/ios/ui-scrolling-tree.html:
+
+2019-10-21  Simon Fraser  <simon.fraser@apple.com>
+
         [iOS WK2] Support hiding iframe scrollbars via ::-webkit-scrollbar style
         https://bugs.webkit.org/show_bug.cgi?id=203178
 
index bf9f96b..0d5ae99 100644 (file)
@@ -63,6 +63,3 @@ media/video-concurrent-visible-playback.html [ Pass Timeout ]
 
 # <rdar://problem/52914585> (Layout Test fast/forms/file/file-input-reset-using-open-panel.html is Failing)
 fast/forms/file/file-input-reset-using-open-panel.html [ ImageOnlyFailure ]
-
-# rdar://52970947 (Layout Test scrollingcoordinator/ios/ui-scrolling-tree.html is a Flaky Failure on iPad iOS 13)
-scrollingcoordinator/ios/ui-scrolling-tree.html [ Pass Failure ]
index a7b53cf..2905a9a 100644 (file)
@@ -1,7 +1,7 @@
 
 (scrolling tree
   (frame scrolling node
-    (scrollable area size width=1280 height=1673)
+    (scrollable area size width=768 height=1004)
     (total content size width=1308 height=2021)
     (last committed scroll position (0,0))
     (scrollable area parameters 
@@ -9,12 +9,12 @@
       (vertical scroll elasticity 1)
       (horizontal scrollbar mode 0)
       (vertical scrollbar mode 0))
-    (layout viewport (0,0) width=1280 height=1673)
+    (layout viewport (0,0) width=768 height=1004)
     (min layoutViewport origin (0,0))
-    (max layoutViewport origin (28,348))
+    (max layoutViewport origin (540,1017))
     (behavior for fixed 0)
     (fixed node
       (fixed constraints 
-        (viewport-rect-at-last-layout (0,0) width=1280 height=1673)
+        (viewport-rect-at-last-layout (0,0) width=768 height=1004)
         (layer-position-at-last-layout (12,10)))
       (layer top left (12,10)))))
index 4ad0413..fcd332b 100644 (file)
@@ -1,4 +1,4 @@
-<!DOCTYPE html> <!-- webkit-test-runner [ useFlexibleViewport=true ] -->
+<!DOCTYPE html> <!-- webkit-test-runner [ useFlexibleViewport=true contentMode=mobile ] -->
 <html>
 <head>
     <meta name="viewport" content="initial-scale=1.0">
             background-color: blue;
         }
     </style>
+    <script src="../../resources/ui-helper.js"></script>
     <script>
         if (window.testRunner) {
             testRunner.waitUntilDone();
             testRunner.dumpAsText();
         }
 
-        function getScrollingTreeUIScript()
+        async function doTest()
         {
-            return `(function() {
-                return uiController.scrollingTreeAsText;
-            })();`;
-        }
-
-        function doTest()
-        {
-            if (!testRunner.runUIScript)
-                return
-
-            testRunner.runUIScript(getScrollingTreeUIScript(), function(scrollingTree) {
-                document.getElementById('layers').textContent = scrollingTree;
-                testRunner.notifyDone();
-            });
+            await UIHelper.ensureStablePresentationUpdate();
+            let scrollingTree = await UIHelper.getScrollingTree();
+            document.getElementById('layers').textContent = scrollingTree;
+            testRunner.notifyDone();
         }
         
         window.addEventListener('load', doTest, false);