LayoutTest fast/events/currentTarget-gc-crash.html is a flaky failure
authorcdumez@apple.com <cdumez@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 27 Feb 2017 20:01:09 +0000 (20:01 +0000)
committercdumez@apple.com <cdumez@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 27 Feb 2017 20:01:09 +0000 (20:01 +0000)
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:

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

LayoutTests/ChangeLog
LayoutTests/fast/events/currentTarget-gc-crash.html

index a115693..4ddb0e3 100644 (file)
@@ -1,3 +1,14 @@
+2017-02-27  Chris Dumez  <cdumez@apple.com>
+
+        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  <graouts@apple.com>
 
         [Modern Media Controls] Status text does not appear on iOS
index 507da9b..e2b9001 100644 (file)
@@ -9,6 +9,7 @@ function f1() {
     window.frames.event = window.event;
     gc();
 }
+let startedTimer = false;
 function f2() {
     var h = new XMLHttpRequest();
     h.onreadystatechange = f1;
@@ -18,7 +19,11 @@ function f2() {
     try {
         e.currentTarget.click();
     } catch(e) { }
-    setTimeout(finishJSTest, 100);
+
+    if (!startedTimer) {
+        setTimeout(finishJSTest, 100);
+        startedTimer = true;
+    }
 }
 </script>
 </head>