fast/workers/worker-user-gesture.html crashes on iOS

https://bugs.webkit.org/show_bug.cgi?id=174592

Reviewed by Simon Fraser.

Make sure the tap/click event is fully processed before finishing the test by using
the promise returned by UIHelper.activateAt().

* fast/workers/worker-user-gesture.html:

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

diff --git a/LayoutTests/ChangeLog b/LayoutTests/ChangeLog
index 1c7972a932dea78edb8c6abd0d7da1b1b659d3c8..8758c6bdb636a307a13e6010a310e65872845fe2 100644 (file)
--- a/LayoutTests/ChangeLog
+++ b/LayoutTests/ChangeLog
@@ -1,3 +1,15 @@
+2017-07-17  Chris Dumez  <cdumez@apple.com>
+
+        fast/workers/worker-user-gesture.html crashes on iOS
+        https://bugs.webkit.org/show_bug.cgi?id=174592
+
+        Reviewed by Simon Fraser.
+
+        Make sure the tap/click event is fully processed before finishing the test by using
+        the promise returned by UIHelper.activateAt().
+
+        * fast/workers/worker-user-gesture.html:
+
 2017-07-17  Simon Fraser  <simon.fraser@apple.com>
 
         clientX/clientY on TouchEvent.touches are wrong

diff --git a/LayoutTests/fast/workers/worker-user-gesture.html b/LayoutTests/fast/workers/worker-user-gesture.html
index 65ba863db0be8904ac7a08f448c5cbab5ece37ac..cc5bf5463793d484ec140afbf84785af65e8b03a 100644 (file)
--- a/LayoutTests/fast/workers/worker-user-gesture.html
+++ b/LayoutTests/fast/workers/worker-user-gesture.html
@@ -18,12 +18,13 @@ function doUserGesture() {
     let inputElement = document.getElementById("testButton");
     let centerX = inputElement.offsetLeft + inputElement.offsetWidth / 2;
     let centerY = inputElement.offsetTop + inputElement.offsetHeight / 2;
-    UIHelper.activateAt(centerX, centerY);
-    if (++gestureCount == 10) {
-        finishJSTest();
-        return;
-    }
-    setTimeout(doUserGesture, 0);
+    UIHelper.activateAt(centerX, centerY).then(function() {
+        if (++gestureCount == 10) {
+            finishJSTest();
+            return;
+        }
+        setTimeout(doUserGesture, 0);
+    });
 }
 
 setTimeout(doUserGesture, 0);