REGRESSION: (r244182) Layout Test editing/execCommand/insert-nested-lists.html is...
authorrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 19 Apr 2019 19:33:23 +0000 (19:33 +0000)
committerrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 19 Apr 2019 19:33:23 +0000 (19:33 +0000)
https://bugs.webkit.org/show_bug.cgi?id=196985

Reviewed by Wenson Hsieh.

The flakiness was caused by the test incorrectly assuming that rAF would
be enough to close the undo group in the UI process.

Fixed the test by explicitly waiting for the runloop in the ui process to run
by executing a UI script.

* editing/execCommand/insert-nested-lists.html:

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

LayoutTests/ChangeLog
LayoutTests/editing/execCommand/insert-nested-lists.html

index 8eb01a2..f55f3b8 100644 (file)
@@ -1,3 +1,18 @@
+2019-04-19  Ryosuke Niwa  <rniwa@webkit.org>
+
+        REGRESSION: (r244182) Layout Test editing/execCommand/insert-nested-lists.html is flaky
+        https://bugs.webkit.org/show_bug.cgi?id=196985
+
+        Reviewed by Wenson Hsieh.
+
+        The flakiness was caused by the test incorrectly assuming that rAF would
+        be enough to close the undo group in the UI process.
+
+        Fixed the test by explicitly waiting for the runloop in the ui process to run
+        by executing a UI script.
+
+        * editing/execCommand/insert-nested-lists.html:
+
 2019-04-18  Megan Gardner  <megan_gardner@apple.com>
 
         Update autoscroll test to lift at the end and be less flakey
index bd4f34b..fe068be 100644 (file)
 
     // By default, AppKit on macOS coalesces undo operations that occur within the same runloop. Wait until the next
     // runloop before inserting another unordered list to ensure that it gets its own entry in the undo stack.
-    await new Promise(resolve => requestAnimationFrame(resolve));
+    await new Promise((resolve) => {
+        if (window.testRunner)
+            testRunner.runUIScript(`(function() { uiController.uiScriptComplete(); })()`, resolve);
+        else
+            setTimeout(resolve, 500);
+    });
 
     document.execCommand("InsertNestedUnorderedList");
     Markup.dump("editor", "After the second unordered list insertion");