fast/fixed-layout/fixed-layout.html is flaky
authortimothy_horton@apple.com <timothy_horton@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 4 Apr 2015 00:07:54 +0000 (00:07 +0000)
committertimothy_horton@apple.com <timothy_horton@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 4 Apr 2015 00:07:54 +0000 (00:07 +0000)
https://bugs.webkit.org/show_bug.cgi?id=143379

Reviewed by Simon Fraser.

* TestExpectations:
Un-flakify the fixed layout test.

* fast/css-grid-layout/flex-content-sized-columns-resize-expected.html:
Remove a mistaken line in the original test; this function doesn't exist in WebKit
and was presumably ported directly from the Blink test (and then never noticed because
JS exceptions thrown in -expected files go nowhere).

* fast/fixed-layout/fixed-layout.html:
Make the fixed layout test less racy by waiting for the resize event (which can be
very asynchronous). Also, force a layout before resizing to ensure that we
always do get a resize event.

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

LayoutTests/ChangeLog
LayoutTests/TestExpectations
LayoutTests/fast/css-grid-layout/flex-content-sized-columns-resize-expected.html
LayoutTests/fast/fixed-layout/fixed-layout.html

index 3f6ea7a..414c77a 100644 (file)
@@ -1,3 +1,23 @@
+2015-04-03  Tim Horton  <timothy_horton@apple.com>
+
+        fast/fixed-layout/fixed-layout.html is flaky
+        https://bugs.webkit.org/show_bug.cgi?id=143379
+
+        Reviewed by Simon Fraser.
+
+        * TestExpectations:
+        Un-flakify the fixed layout test.
+
+        * fast/css-grid-layout/flex-content-sized-columns-resize-expected.html:
+        Remove a mistaken line in the original test; this function doesn't exist in WebKit
+        and was presumably ported directly from the Blink test (and then never noticed because
+        JS exceptions thrown in -expected files go nowhere).
+
+        * fast/fixed-layout/fixed-layout.html:
+        Make the fixed layout test less racy by waiting for the resize event (which can be
+        very asynchronous). Also, force a layout before resizing to ensure that we
+        always do get a resize event.
+
 2015-04-03  Chris Fleizach  <cfleizach@apple.com>
 
         AX: Toggling check box state not speaking using plain space, not VO space.
index f0b951c..9ce73a4 100644 (file)
@@ -515,5 +515,3 @@ webkit.org/b/142937 ietestcenter/Javascript/15.2.3.13-1.html [ Failure ]
 webkit.org/b/142937 ietestcenter/Javascript/15.2.3.14-1-1.html [ Failure ]
 webkit.org/b/142937 ietestcenter/Javascript/15.2.3.14-1-2.html [ Failure ]
 webkit.org/b/142937 ietestcenter/Javascript/15.2.3.14-1-3.html [ Failure ]
-
-webkit.org/b/143379 fast/fixed-layout/fixed-layout.html [ Pass Failure ]
index f911761..7372d81 100644 (file)
@@ -20,7 +20,6 @@
     <script>
         function runTest() {
             if (window.testRunner) {
-                testRunner.useUnfortunateSynchronousResizeMode();
                 window.resizeTo(600, 600);
             }
         }
index 37a0414..d0f3eb5 100644 (file)
@@ -6,16 +6,19 @@ window.onload = function () {
         return;
     }
 
-    testRunner.waitUntilDone();
+    // Ensure layout is up to date here so that we get a resize event instead of just using
+    // 200x200 as the initial size, in WebKit1 (causing the test to hang).
+    document.body.offsetTop;
 
+    testRunner.waitUntilDone();
     window.resizeTo(200, 200);
+}
 
+window.onresize = function () {
     internals.setUseFixedLayout(true);
     internals.setFixedLayoutSize(400, 400);
 
-    setTimeout(function () {
-        testRunner.notifyDone();
-    }, 0);
+    testRunner.notifyDone();
 }
 </script>
 </head>