[webkit-reviews] review granted: [Bug 179778] LayoutTest http/tests/workers/service/registration-clear-redundant-worker.html is a flaky failure : [Attachment 327928] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Nov 29 16:58:59 PST 2017


youenn fablet <youennf at gmail.com> has granted Chris Dumez <cdumez at apple.com>'s
request for review:
Bug 179778: LayoutTest
http/tests/workers/service/registration-clear-redundant-worker.html is a flaky
failure
https://bugs.webkit.org/show_bug.cgi?id=179778

Attachment 327928: Patch

https://bugs.webkit.org/attachment.cgi?id=327928&action=review




--- Comment #9 from youenn fablet <youennf at gmail.com> ---
Comment on attachment 327928
  --> https://bugs.webkit.org/attachment.cgi?id=327928
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=327928&action=review

>
LayoutTests/http/tests/workers/service/registration-clear-redundant-worker.html
:28
> +		   handle = setInterval(function() {

Maybe we could add a limit to the number of retries to allow gracefully failing
instead of timing out.
Something like 100 tries every 10 ms for instance.


More information about the webkit-reviews mailing list