Changeset 255223 in webkit

Timestamp:

Jan 27, 2020, 7:54:59 PM (6 years ago)

Author:

rniwa@webkit.org

Message:

http/tests/loading/remove-child-triggers-parser.html is flaky
​https://bugs.webkit.org/show_bug.cgi?id=103115

Reviewed by Alexey Proskuryakov.

Delay the end of the test until the timer fires.

• http/tests/loading/remove-child-triggers-parser-expected.txt:
• http/tests/loading/remove-child-triggers-parser.html:

Location:

trunk/LayoutTests

Files:

• ChangeLog (modified) (1 diff)
• http/tests/loading/remove-child-triggers-parser-expected.txt (modified) (1 diff)
• http/tests/loading/remove-child-triggers-parser.html (modified) (1 diff)

trunk/LayoutTests/ChangeLog

@@ +1 @@
+2020-01-27  Ryosuke Niwa  <rniwa@webkit.org>
+
+        http/tests/loading/remove-child-triggers-parser.html is flaky
+        https://bugs.webkit.org/show_bug.cgi?id=103115
+
+        Reviewed by Alexey Proskuryakov.
+
+        Delay the end of the test until the timer fires.
+
+        * http/tests/loading/remove-child-triggers-parser-expected.txt:
+        * http/tests/loading/remove-child-triggers-parser.html:
+
 2020-01-27  Peng Liu  <peng.liu6@apple.com>
 

trunk/LayoutTests/http/tests/loading/remove-child-triggers-parser-expected.txt

@@ -4 +4 @@
 main frame - didHandleOnloadEventsForFrame
 main frame - didFinishLoadForFrame
-

trunk/LayoutTests/http/tests/loading/remove-child-triggers-parser.html

@@ -2 +2 @@
 <!-- This test covers Bug 93641 -->
 <script>
-if (window.testRunner)
+if (window.testRunner) {
     testRunner.dumpAsText();
-setTimeout(function() { var child = document.documentElement; child.parentNode.removeChild(child); }, 36);
+    testRunner.waitUntilDone();
+}
+window.onload = () => {
+    setTimeout(function() {
+        var child = document.documentElement;
+        child.parentNode.removeChild(child);
+        if (window.testRunner)
+            testRunner.notifyDone();
+    }, 36);
+}
 </script>
 <script></script>