Title: [88746] trunk/LayoutTests
Revision
88746
Author
[email protected]
Date
2011-06-13 18:15:27 -0700 (Mon, 13 Jun 2011)

Log Message

2011-06-13  David Levin  <[email protected]>

        Reviewed by Dmitry Titov.

        fast/workers/worker-close.html is flaky on slower machines.
        https://bugs.webkit.org/show_bug.cgi?id=62609

        * fast/workers/worker-close.html: Move a setTimeout to start after
        the response message is received to make the ordering of messages
        deterministic.

Modified Paths

Diff

Modified: trunk/LayoutTests/ChangeLog (88745 => 88746)


--- trunk/LayoutTests/ChangeLog	2011-06-14 01:13:56 UTC (rev 88745)
+++ trunk/LayoutTests/ChangeLog	2011-06-14 01:15:27 UTC (rev 88746)
@@ -1,3 +1,14 @@
+2011-06-13  David Levin  <[email protected]>
+
+        Reviewed by Dmitry Titov.
+
+        fast/workers/worker-close.html is flaky on slower machines.
+        https://bugs.webkit.org/show_bug.cgi?id=62609
+
+        * fast/workers/worker-close.html: Move a setTimeout to start after
+        the response message is received to make the ordering of messages
+        deterministic.
+
 2011-06-13  Ryosuke Niwa  <[email protected]>
 
         Chromium Mac test expectation updates; Removed failing test expectations from

Modified: trunk/LayoutTests/fast/workers/worker-close.html (88745 => 88746)


--- trunk/LayoutTests/fast/workers/worker-close.html	2011-06-14 01:13:56 UTC (rev 88745)
+++ trunk/LayoutTests/fast/workers/worker-close.html	2011-06-14 01:15:27 UTC (rev 88746)
@@ -41,14 +41,14 @@
     worker.postMessage("close");
     worker.postMessage("ping");
     worker._onmessage_ = function(evt) {
-        if (evt.data != "pong")
+        if (evt.data != "pong") {
             log("PASS: Received message posted right after close() was invoked: " + evt.data);
-        else {
+            timeout = setTimeout(testErrorAfterClose, 1000);
+        } else {
             log("FAIL: Received a message originated from a handler in the worker after the JS fragment with close() exited" + evt.data);
             done();
         }
     };
-    timeout = setTimeout(testErrorAfterClose, 1000);
 }
 
 function testErrorAfterClose()
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to