Layout Test http/tests/workers/service/basic-unregister-then-register-again-reuse...
authorcdumez@apple.com <cdumez@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 24 Jan 2018 21:26:47 +0000 (21:26 +0000)
committercdumez@apple.com <cdumez@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 24 Jan 2018 21:26:47 +0000 (21:26 +0000)
https://bugs.webkit.org/show_bug.cgi?id=182030
<rdar://problem/36802933>

Reviewed by Youenn Fablet.

The service worker was only extending its lifetime using waitUntil() for 10 seconds. On the flakiness
dashboard, I see that the test sometimes takes more than 10 seconds to run on some bots, which can
cause the test to fail since the service worker is not keeping the registration alive anymore.
To address the issue, raise the lifetime to 30 seconds.

* http/tests/workers/service/resources/basic-unregister-then-register-again-reuse-worker.js:
(event.waitUntil.new.Promise):

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

LayoutTests/ChangeLog
LayoutTests/http/tests/workers/service/resources/basic-unregister-then-register-again-reuse-worker.js

index 83e99b5..628277e 100644 (file)
@@ -1,3 +1,19 @@
+2018-01-24  Chris Dumez  <cdumez@apple.com>
+
+        Layout Test http/tests/workers/service/basic-unregister-then-register-again-reuse.html is flaky
+        https://bugs.webkit.org/show_bug.cgi?id=182030
+        <rdar://problem/36802933>
+
+        Reviewed by Youenn Fablet.
+
+        The service worker was only extending its lifetime using waitUntil() for 10 seconds. On the flakiness
+        dashboard, I see that the test sometimes takes more than 10 seconds to run on some bots, which can
+        cause the test to fail since the service worker is not keeping the registration alive anymore.
+        To address the issue, raise the lifetime to 30 seconds.
+
+        * http/tests/workers/service/resources/basic-unregister-then-register-again-reuse-worker.js:
+        (event.waitUntil.new.Promise):
+
 2018-01-24  Alex Christensen  <achristensen@webkit.org>
 
         Remove El Capitan test expectations.
index 4b79afd..a4fb76a 100644 (file)
@@ -2,7 +2,7 @@ self.addEventListener("message", function(event) {
     event.waitUntil(new Promise((resolve, reject) => {
         setTimeout(() => {
             resolve();
-        }, 10000);
+        }, 30000);
     }));
     client = event.source;
     setTimeout(function() {