Bug 1228277 P3 Remove skipWaiting() work around from the wpt test. r=asuth
authorBen Kelly <ben@wanderview.com>
Wed, 27 Jul 2016 19:09:15 -0400
changeset 306964 2a326dd181aa25196bb7182d601be0d8027b93a6
parent 306963 d4e7fea164eb74d8750c42d9bc188ab3984e9658
child 306965 de70d037b6b908ff34677dcdbdb525492609846e
push id30502
push usercbook@mozilla.com
push dateThu, 28 Jul 2016 15:43:16 +0000
treeherdermozilla-central@9ec789c0ee5b [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersasuth
bugs1228277
milestone50.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1228277 P3 Remove skipWaiting() work around from the wpt test. r=asuth
testing/web-platform/tests/service-workers/service-worker/resources/skip-waiting-worker.js
--- a/testing/web-platform/tests/service-workers/service-worker/resources/skip-waiting-worker.js
+++ b/testing/web-platform/tests/service-workers/service-worker/resources/skip-waiting-worker.js
@@ -1,16 +1,12 @@
 importScripts('worker-testharness.js');
 
 promise_test(function() {
-    // wait for the worker to reach "installing" state, otherwise skipWaiting()
-    // will fail. Bug 1228277
-    return new Promise(function(res, rej) {
-      oninstall = res;
-    }).then(() => skipWaiting())
+    return skipWaiting()
       .then(function(result) {
           assert_equals(result, undefined,
                         'Promise should be resolved with undefined');
         })
       .then(function() {
           var promises = [];
           for (var i = 0; i < 8; ++i)
             promises.push(self.skipWaiting());