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 346994 2a326dd181aa25196bb7182d601be0d8027b93a6
parent 346993 d4e7fea164eb74d8750c42d9bc188ab3984e9658
child 346995 de70d037b6b908ff34677dcdbdb525492609846e
push id6389
push userraliiev@mozilla.com
push dateMon, 19 Sep 2016 13:38:22 +0000
treeherdermozilla-beta@01d67bfe6c81 [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());