fast/events/scroll-after-click-on-tab-index.html is flaky
authorap@apple.com <ap@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 8 Oct 2015 17:44:18 +0000 (17:44 +0000)
committerap@apple.com <ap@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 8 Oct 2015 17:44:18 +0000 (17:44 +0000)
https://bugs.webkit.org/show_bug.cgi?id=149859

Reviewed by Simon Fraser.

* fast/events/scroll-after-click-on-tab-index-expected.txt:
* fast/events/scroll-after-click-on-tab-index.html:
* platform/mac/TestExpectations:

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

LayoutTests/ChangeLog
LayoutTests/fast/events/scroll-after-click-on-tab-index-expected.txt
LayoutTests/fast/events/scroll-after-click-on-tab-index.html
LayoutTests/platform/mac/TestExpectations

index c1a7033..b453f27 100644 (file)
@@ -1,3 +1,14 @@
+2015-10-08  Alexey Proskuryakov  <ap@apple.com>
+
+        fast/events/scroll-after-click-on-tab-index.html is flaky
+        https://bugs.webkit.org/show_bug.cgi?id=149859
+
+        Reviewed by Simon Fraser.
+
+        * fast/events/scroll-after-click-on-tab-index-expected.txt:
+        * fast/events/scroll-after-click-on-tab-index.html:
+        * platform/mac/TestExpectations:
+
 2015-10-08  Jiewen Tan  <jiewen_tan@apple.com>
 
         Add NULL check for renderBox::layer() on applying zoom level change
index 3222b68..193d814 100644 (file)
@@ -1,10 +1,9 @@
 ALERT: Scroll position 0
 Click the text below and then try to scroll.
+PASS Scroll position is more than 0
 PASS successfullyParsed is true
 
 TEST COMPLETE
-Scroll position is more than 0
-PASS document.body.scrollTop > 0 is true
 
 
 
index 7a09f3c..4cbf154 100644 (file)
@@ -1,5 +1,5 @@
 <html>
-<script src="../../resources/js-test-pre.js"></script>
+<script src="../../resources/js-test.js"></script>
 <script>
 function getCenterFor(element)
 {
@@ -14,8 +14,7 @@ function runTest()
   if (!window.eventSender)
     return;
 
-  testRunner.waitUntilDone();
-  testRunner.dumpAsText();
+  jsTestIsAsync = true;
 
   alert("Scroll position " + document.body.scrollTop);
   center = getCenterFor(document.getElementById("sp"));
@@ -23,14 +22,23 @@ function runTest()
   eventSender.mouseDown();
   eventSender.mouseUp();
   eventSender.keyDown("downArrow");
-  setTimeout(verifyScrolling, 15);
+  intervalID = setInterval(function() {
+    if (document.body.scrollTop > 0) {
+      testPassed("Scroll position is more than 0");
+      done();
+    }
+  }, 10);
+  timeoutID = setTimeout(function() {
+    testFailed("Scroll position was still 10 after 20 seconds");
+    done();
+  }, 20000);
 }
 
-function verifyScrolling()
+function done()
 {
-  debug("Scroll position is more than 0");
-  shouldBe("document.body.scrollTop > 0", "true");
-  testRunner.notifyDone();
+  clearInterval(intervalID);
+  clearTimeout(timeoutID);
+  finishJSTest();
 }
 
 </script>
@@ -39,7 +47,6 @@ function verifyScrolling()
 <body onload="runTest();">
 <input type="radio" id="myradio">Click the text below and then try to scroll.<br>
 <div id="console"></div>
-<script src="../../resources/js-test-post.js"></script>
 <br><br><br><br><br><br><br><br><br><br>
 <span tabindex=1 id="sp">My span</span>
 <br><br><br><br><br><br><br><br><br><br><br><br><br><br><br><br><br><br><br><br><br><br><br><br><br><br>
index 597028f..2463fd9 100644 (file)
@@ -1334,6 +1334,3 @@ webkit.org/b/149441 fast/shadow-dom/css-scoping-shadow-slot-display-override.htm
 
 # Marking test as flaky in El Capitan
 webkit.org/b/149819 [ Debug ElCapitan+ ] compositing/video/video-poster.html [ Pass Crash ]
-
-# Marking test as flaky
-webkit.org/b/149859 fast/events/scroll-after-click-on-tab-index.html [ Pass Failure ]