swipe-start-hysteresis-failures is flaky
authortimothy_horton@apple.com <timothy_horton@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 4 Sep 2015 19:24:55 +0000 (19:24 +0000)
committertimothy_horton@apple.com <timothy_horton@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 4 Sep 2015 19:24:55 +0000 (19:24 +0000)
https://bugs.webkit.org/show_bug.cgi?id=148809

Reviewed by Alexey Proskuryakov.

* swipe/swipe-start-hysteresis-failures.html:
* swipe/swipe-start-hysteresis-failures-expected.txt:
Instead of depending on the specific timing of didStartSwipe vs. the events
in this case, just make sure we hit it by the time the test finishes.

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

LayoutTests/ChangeLog
LayoutTests/swipe/swipe-start-hysteresis-failures-expected.txt
LayoutTests/swipe/swipe-start-hysteresis-failures.html

index 25ab259..26aad84 100644 (file)
@@ -1,3 +1,15 @@
+2015-09-04  Tim Horton  <timothy_horton@apple.com>
+
+        swipe-start-hysteresis-failures is flaky
+        https://bugs.webkit.org/show_bug.cgi?id=148809
+
+        Reviewed by Alexey Proskuryakov.
+
+        * swipe/swipe-start-hysteresis-failures.html:
+        * swipe/swipe-start-hysteresis-failures-expected.txt:
+        Instead of depending on the specific timing of didStartSwipe vs. the events
+        in this case, just make sure we hit it by the time the test finishes.
+
 2015-09-04  Alexey Proskuryakov  <ap@apple.com>
 
         Skip newly imported IndexedDB test on iOS WebKit1, because IndexedDB only works with WebKit2.
index 02e4f7b..5a480a9 100644 (file)
@@ -17,7 +17,6 @@ scroll event (delta 0 0, phase 'ended')
 swipe event (delta 0 0, phase 'maybegin')
 scroll event (delta 3 1, phase 'began')
 scroll event (delta 0 0, phase 'changed')
-didBeginSwipe
 scroll event (delta 0 0, phase 'ended')
 swipe event (delta 0 0, phase 'ended')
 
index 6f2df63..65817d3 100644 (file)
@@ -59,14 +59,15 @@ function doRegularSwipeGesture()
 {
     // Swap in a different callback that makes sure we *do* reach didBeginSwipe.
     testRunner.clearTestRunnerCallbacks();
+    var sawDidBeginSwipe = false;
     testRunner.installDidBeginSwipeCallback(function () {
-        log("didBeginSwipe");
-
+        sawDidBeginSwipe = true;
         // Need a swipe-end event to clean up.
         eventQueue.enqueueSwipeEvent(0, 0, 'ended');
     });
 
     testRunner.installDidRemoveSwipeSnapshotCallback(function () {
+        shouldBe(true, sawDidBeginSwipe, "The last gesture should start a swipe, because it is sufficient magnitude and in the correct direction.")
         testComplete();
     });