Bug 1228277 - Part 3: Remove skipWaiting() work around from the wpt test. r=asuth, a=sledru
authorBen Kelly <ben@wanderview.com>
Wed, 27 Jul 2016 19:09:15 -0400
changeset 342154 a850b3b58cb13c79b91a1376af5edae53f1b6000
parent 342153 a4aeae74f3ca866d348c057717e28304e288f115
child 342155 9606affd4ad0de44efea712b7e6256502b584207
push id1183
push userraliiev@mozilla.com
push dateMon, 05 Sep 2016 20:01:49 +0000
treeherdermozilla-release@3148731bed45 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersasuth, sledru
bugs1228277
milestone49.0a2
Bug 1228277 - Part 3: Remove skipWaiting() work around from the wpt test. r=asuth, a=sledru
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());