Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(6417)

Unified Diff: chrome/test/data/extensions/api_test/service_worker/api_worker_ref_count/page.js

Issue 2166523003: Add ref count to service workers for extension API. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: address comments from Devlin & MichaelN Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/test/data/extensions/api_test/service_worker/api_worker_ref_count/page.js
diff --git a/chrome/test/data/extensions/api_test/service_worker/tabs_create/page.js b/chrome/test/data/extensions/api_test/service_worker/api_worker_ref_count/page.js
similarity index 55%
copy from chrome/test/data/extensions/api_test/service_worker/tabs_create/page.js
copy to chrome/test/data/extensions/api_test/service_worker/api_worker_ref_count/page.js
index 0542b5485544a0c5623b18b44e64b0f1cbdb71ca..3f8ebcc07382c5efbc4d7084d91d66101f70708f 100644
--- a/chrome/test/data/extensions/api_test/service_worker/tabs_create/page.js
+++ b/chrome/test/data/extensions/api_test/service_worker/api_worker_ref_count/page.js
@@ -2,29 +2,21 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-var serviceWorkerPromise = new Promise(function(resolve, reject) {
+window.runServiceWorker = function() {
navigator.serviceWorker.register('sw.js').then(function() {
return navigator.serviceWorker.ready;
}).then(function(registration) {
var sw = registration.active;
var channel = new MessageChannel();
channel.port1.onmessage = function(e) {
- if (e.data == 'chrome.tabs.create callback') {
- resolve(e.data);
+ if (e.data == 'Worker reply: Hello world') {
+ chrome.test.sendMessage('SUCCESS');
} else {
- reject(e.data);
+ chrome.test.sendMessage('FAILURE');
}
};
- sw.postMessage('createTab', [channel.port2]);
- }).catch(function(err) {
- reject(err);
- });
-});
-
-window.runServiceWorker = function() {
- serviceWorkerPromise.then(function(message) {
- window.domAutomationController.send(message);
+ sw.postMessage('sendMessageTest', [channel.port2]);
}).catch(function(err) {
- window.domAutomationController.send('FAILURE');
+ chrome.test.sendMessage('FAILURE');
});
};

Powered by Google App Engine
This is Rietveld 408576698