Changeset 57916 in webkit

Timestamp:

Apr 20, 2010 2:12:33 PM (14 years ago)

Author:

dimich@chromium.org

Message:

worker-close.html is flaky on slow machines.
​https://bugs.webkit.org/show_bug.cgi?id=37887

Reviewed by Adam Barth.

• fast/workers/worker-close.html: Move the start of the timer to terminate the test after an expected message is received.

Location:

trunk/LayoutTests

Files:

• ChangeLog (modified) (1 diff)
• fast/workers/worker-close.html (modified) (1 diff)

trunk/LayoutTests/ChangeLog

@@ +1 @@
+2010-04-20  Dmitry Titov  <dimich@chromium.org>
+
+        Reviewed by Adam Barth.
+
+        worker-close.html is flaky on slow machines.
+        https://bugs.webkit.org/show_bug.cgi?id=37887
+
+        * fast/workers/worker-close.html: Move the start of the timer to terminate the test after an expected message is received.
+
 2010-04-20  James Robinson  <jamesr@chromium.org>
 

trunk/LayoutTests/fast/workers/worker-close.html

@@ -88 +88 @@
     worker.onmessage = function(evt) {
         log("PASS: Received message after worker closed: " + evt.data);
+        // Give worker a chance to close first, then terminate it.
+        timeout = setTimeout(function() {
+            worker.terminate();
+            done();
+        }, 500);
     };
-    // Give worker a chance to close first, then terminate it.
-    timeout = setTimeout(function() {
-        worker.terminate();
-        done();
-    }, 500);
 }