Title: [213085] trunk/LayoutTests
Revision
213085
Author
[email protected]
Date
2017-02-27 12:01:09 -0800 (Mon, 27 Feb 2017)

Log Message

LayoutTest fast/events/currentTarget-gc-crash.html is a flaky failure
https://bugs.webkit.org/show_bug.cgi?id=168917

Reviewed by Alex Christensen.

Make sure we never call finishJSTest() more than once.

* fast/events/currentTarget-gc-crash.html:

Modified Paths

Diff

Modified: trunk/LayoutTests/ChangeLog (213084 => 213085)


--- trunk/LayoutTests/ChangeLog	2017-02-27 19:55:09 UTC (rev 213084)
+++ trunk/LayoutTests/ChangeLog	2017-02-27 20:01:09 UTC (rev 213085)
@@ -1,3 +1,14 @@
+2017-02-27  Chris Dumez  <[email protected]>
+
+        LayoutTest fast/events/currentTarget-gc-crash.html is a flaky failure
+        https://bugs.webkit.org/show_bug.cgi?id=168917
+
+        Reviewed by Alex Christensen.
+
+        Make sure we never call finishJSTest() more than once.
+
+        * fast/events/currentTarget-gc-crash.html:
+
 2017-02-27  Antoine Quint  <[email protected]>
 
         [Modern Media Controls] Status text does not appear on iOS

Modified: trunk/LayoutTests/fast/events/currentTarget-gc-crash.html (213084 => 213085)


--- trunk/LayoutTests/fast/events/currentTarget-gc-crash.html	2017-02-27 19:55:09 UTC (rev 213084)
+++ trunk/LayoutTests/fast/events/currentTarget-gc-crash.html	2017-02-27 20:01:09 UTC (rev 213085)
@@ -9,6 +9,7 @@
     window.frames.event = window.event;
     gc();
 }
+let startedTimer = false;
 function f2() {
     var h = new XMLHttpRequest();
     h._onreadystatechange_ = f1;
@@ -18,7 +19,11 @@
     try {
         e.currentTarget.click();
     } catch(e) { }
-    setTimeout(finishJSTest, 100);
+
+    if (!startedTimer) {
+        setTimeout(finishJSTest, 100);
+        startedTimer = true;
+    }
 }
 </script>
 </head>
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to