For example, http://build.chromium.org/p/chromium/builders/Mac10.5%20Tests%20(1)/builds/8391/steps/ui_tests/logs/stdio
Created attachment 97036 [details] Patch
Comment on attachment 97036 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=97036&action=review > LayoutTests/fast/workers/worker-close.html:46 > + timeout = setTimeout(testErrorAfterClose, 1000); Do we need a 1000ms delay now?
(In reply to comment #2) > (From update of attachment 97036 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=97036&action=review > > > LayoutTests/fast/workers/worker-close.html:46 > > + timeout = setTimeout(testErrorAfterClose, 1000); > > Do we need a 1000ms delay now? The delay wasn't about waiting for this response. It was about waiting to see if it got back a response to the "worker.postMessage("ping");" message.
Comment on attachment 97036 [details] Patch Thanks!
Comment on attachment 97036 [details] Patch Clearing flags on attachment: 97036 Committed r88746: <http://trac.webkit.org/changeset/88746>
All reviewed patches have been landed. Closing bug.