diff --git a/dom/push/test/xpcshell/head.js b/dom/push/test/xpcshell/head.js index ac312b5ab83b..6d77d0cb63e5 100644 --- a/dom/push/test/xpcshell/head.js +++ b/dom/push/test/xpcshell/head.js @@ -168,23 +168,6 @@ function makeStub(target, stubs) { }); } -/** - * Disables `push` and `pushsubscriptionchange` service worker events for the - * given scopes. These events cause crashes in xpcshell, so we disable them - * for testing nsIPushNotificationService. - * - * @param {String[]} scopes A list of scope URLs. - */ -function disableServiceWorkerEvents(...scopes) { - for (let scope of scopes) { - Services.perms.add( - Services.io.newURI(scope, null, null), - 'desktop-notification', - Ci.nsIPermissionManager.DENY_ACTION - ); - } -} - /** * Sets default PushService preferences. All pref names are prefixed with * `dom.push.`; any additional preferences will override the defaults. @@ -221,6 +204,8 @@ function setPrefs(prefs = {}) { 'http2.reset_retry_count_after_ms': 60000, maxQuotaPerSubscription: 16, quotaUpdateDelay: 3000, + 'testing.notifyWorkers': false, + 'testing.notifyAllObservers': true, }, prefs); for (let pref in defaultPrefs) { servicePrefs.set(pref, defaultPrefs[pref]); diff --git a/dom/push/test/xpcshell/test_clear_origin_data.js b/dom/push/test/xpcshell/test_clear_origin_data.js index 6b1d430d0055..475ef3cea95a 100644 --- a/dom/push/test/xpcshell/test_clear_origin_data.js +++ b/dom/push/test/xpcshell/test_clear_origin_data.js @@ -41,9 +41,6 @@ function run_test() { requestTimeout: 1000, retryBaseInterval: 150 }); - disableServiceWorkerEvents( - 'https://example.org/1' - ); run_next_test(); } diff --git a/dom/push/test/xpcshell/test_notification_ack.js b/dom/push/test/xpcshell/test_notification_ack.js index bf283542990b..dbb695483998 100644 --- a/dom/push/test/xpcshell/test_notification_ack.js +++ b/dom/push/test/xpcshell/test_notification_ack.js @@ -10,11 +10,6 @@ var userAgentID = '5ab1d1df-7a3d-4024-a469-b9e1bb399fad'; function run_test() { do_get_profile(); setPrefs({userAgentID}); - disableServiceWorkerEvents( - 'https://example.org/1', - 'https://example.org/2', - 'https://example.org/3' - ); run_next_test(); } @@ -50,11 +45,9 @@ add_task(function* test_notification_ack() { yield db.put(record); } - let notifyPromise = Promise.all([ - promiseObserverNotification('push-notification'), - promiseObserverNotification('push-notification'), - promiseObserverNotification('push-notification') - ]); + let notifyCount = 0; + let notifyPromise = promiseObserverNotification('push-message', () => + ++notifyCount == 3); let acks = 0; let ackDone; diff --git a/dom/push/test/xpcshell/test_notification_data.js b/dom/push/test/xpcshell/test_notification_data.js index 92b231ef28a4..b49b246a19a1 100644 --- a/dom/push/test/xpcshell/test_notification_data.js +++ b/dom/push/test/xpcshell/test_notification_data.js @@ -218,9 +218,9 @@ add_task(function* test_notification_ack_data() { ]; let sendAndReceive = testData => { - let messageReceived = promiseObserverNotification('push-notification', (subject, data) => { - let notification = subject.QueryInterface(Ci.nsIPushObserverNotification); - equal(notification.data, testData.receive.data, + let messageReceived = promiseObserverNotification('push-message', (subject, data) => { + let notification = subject.QueryInterface(Ci.nsIPushMessage); + equal(notification.text(), testData.receive.data, 'Check data for notification ' + testData.version); equal(data, testData.receive.scope, 'Check scope for notification ' + testData.version); diff --git a/dom/push/test/xpcshell/test_notification_duplicate.js b/dom/push/test/xpcshell/test_notification_duplicate.js index ed6385f70637..8b97a2393d75 100644 --- a/dom/push/test/xpcshell/test_notification_duplicate.js +++ b/dom/push/test/xpcshell/test_notification_duplicate.js @@ -12,10 +12,6 @@ function run_test() { setPrefs({ userAgentID: userAgentID, }); - disableServiceWorkerEvents( - 'https://example.com/1', - 'https://example.com/2' - ); run_next_test(); } @@ -44,7 +40,7 @@ add_task(function* test_notification_duplicate() { yield db.put(record); } - let notifyPromise = promiseObserverNotification('push-notification'); + let notifyPromise = promiseObserverNotification('push-message'); let acks = 0; let ackDone; diff --git a/dom/push/test/xpcshell/test_notification_error.js b/dom/push/test/xpcshell/test_notification_error.js index 91ce131c8aa3..56a4521f1706 100644 --- a/dom/push/test/xpcshell/test_notification_error.js +++ b/dom/push/test/xpcshell/test_notification_error.js @@ -12,11 +12,6 @@ function run_test() { setPrefs({ userAgentID: userAgentID, }); - disableServiceWorkerEvents( - 'https://example.com/a', - 'https://example.com/b', - 'https://example.com/c' - ); run_next_test(); } @@ -54,16 +49,9 @@ add_task(function* test_notification_error() { yield db.put(record); } - let notifyPromise = Promise.all([ - promiseObserverNotification( - 'push-notification', - (subject, data) => data == 'https://example.com/a' - ), - promiseObserverNotification( - 'push-notification', - (subject, data) => data == 'https://example.com/c' - ) - ]); + let scopes = []; + let notifyPromise = promiseObserverNotification('push-message', (subject, data) => + scopes.push(data) == 2); let ackDone; let ackPromise = new Promise(resolve => ackDone = after(records.length, resolve)); @@ -99,20 +87,15 @@ add_task(function* test_notification_error() { } }); - let [a, c] = yield waitForPromise( + yield waitForPromise( notifyPromise, DEFAULT_TIMEOUT, 'Timed out waiting for notifications' ); - let aPush = a.subject.QueryInterface(Ci.nsIPushObserverNotification); - equal(aPush.pushEndpoint, 'https://example.org/update/success-1', - 'Wrong endpoint for notification A'); - equal(aPush.version, 2, 'Wrong version for notification A'); - - let cPush = c.subject.QueryInterface(Ci.nsIPushObserverNotification); - equal(cPush.pushEndpoint, 'https://example.org/update/success-2', - 'Wrong endpoint for notification C'); - equal(cPush.version, 4, 'Wrong version for notification C'); + ok(scopes.includes('https://example.com/a'), + 'Missing scope for notification A'); + ok(scopes.includes('https://example.com/c'), + 'Missing scope for notification C'); yield waitForPromise(ackPromise, DEFAULT_TIMEOUT, 'Timed out waiting for acknowledgements'); diff --git a/dom/push/test/xpcshell/test_notification_http2.js b/dom/push/test/xpcshell/test_notification_http2.js index a7c03ee7ef2e..44c53c7082d6 100644 --- a/dom/push/test/xpcshell/test_notification_http2.js +++ b/dom/push/test/xpcshell/test_notification_http2.js @@ -35,11 +35,8 @@ function run_test() { prefs.setIntPref("network.http.speculative-parallel-limit", oldPref); - disableServiceWorkerEvents( - 'https://example.com/page/1', - 'https://example.com/page/2', - 'https://example.com/page/3' - ); + servicePrefs.set('testing.notifyWorkers', false); + servicePrefs.set('testing.notifyAllObservers', true); run_next_test(); } @@ -124,24 +121,24 @@ add_task(function* test_pushNotifications() { } let notifyPromise = Promise.all([ - promiseObserverNotification('push-notification', function(subject, data) { - var notification = subject.QueryInterface(Ci.nsIPushObserverNotification); - if (notification && (data == "https://example.com/page/1")){ - equal(subject.data, "Some message", "decoded message is incorrect"); + promiseObserverNotification('push-message', function(subject, data) { + var message = subject.QueryInterface(Ci.nsIPushMessage); + if (message && (data == "https://example.com/page/1")){ + equal(message.text(), "Some message", "decoded message is incorrect"); return true; } }), - promiseObserverNotification('push-notification', function(subject, data) { - var notification = subject.QueryInterface(Ci.nsIPushObserverNotification); - if (notification && (data == "https://example.com/page/2")){ - equal(subject.data, "Some message", "decoded message is incorrect"); + promiseObserverNotification('push-message', function(subject, data) { + var message = subject.QueryInterface(Ci.nsIPushMessage); + if (message && (data == "https://example.com/page/2")){ + equal(message.text(), "Some message", "decoded message is incorrect"); return true; } }), - promiseObserverNotification('push-notification', function(subject, data) { - var notification = subject.QueryInterface(Ci.nsIPushObserverNotification); - if (notification && (data == "https://example.com/page/3")){ - equal(subject.data, "Some message", "decoded message is incorrect"); + promiseObserverNotification('push-message', function(subject, data) { + var message = subject.QueryInterface(Ci.nsIPushMessage); + if (message && (data == "https://example.com/page/3")){ + equal(message.text(), "Some message", "decoded message is incorrect"); return true; } }) diff --git a/dom/push/test/xpcshell/test_notification_incomplete.js b/dom/push/test/xpcshell/test_notification_incomplete.js index ca41ad9e30fc..f78816c62815 100644 --- a/dom/push/test/xpcshell/test_notification_incomplete.js +++ b/dom/push/test/xpcshell/test_notification_incomplete.js @@ -12,12 +12,6 @@ function run_test() { setPrefs({ userAgentID: userAgentID, }); - disableServiceWorkerEvents( - 'https://example.com/page/1', - 'https://example.com/page/2', - 'https://example.com/page/3', - 'https://example.com/page/4' - ); run_next_test(); } @@ -57,9 +51,12 @@ add_task(function* test_notification_incomplete() { yield db.put(record); } - Services.obs.addObserver(function observe(subject, topic, data) { + function observeMessage(subject, topic, data) { ok(false, 'Should not deliver malformed updates'); - }, 'push-notification', false); + } + do_register_cleanup(() => + Services.obs.removeObserver(observeMessage, 'push-message')); + Services.obs.addObserver(observeMessage, 'push-message', false); let notificationDone; let notificationPromise = new Promise(resolve => notificationDone = after(2, resolve)); diff --git a/dom/push/test/xpcshell/test_notification_version_string.js b/dom/push/test/xpcshell/test_notification_version_string.js index c4c7597df585..e55293bf32bd 100644 --- a/dom/push/test/xpcshell/test_notification_version_string.js +++ b/dom/push/test/xpcshell/test_notification_version_string.js @@ -12,9 +12,6 @@ function run_test() { setPrefs({ userAgentID: userAgentID, }); - disableServiceWorkerEvents( - 'https://example.net/case' - ); run_next_test(); } @@ -31,7 +28,7 @@ add_task(function* test_notification_version_string() { systemRecord: true, }); - let notifyPromise = promiseObserverNotification('push-notification'); + let notifyPromise = promiseObserverNotification('push-message'); let ackDone; let ackPromise = new Promise(resolve => ackDone = resolve); @@ -65,11 +62,7 @@ add_task(function* test_notification_version_string() { DEFAULT_TIMEOUT, 'Timed out waiting for string notification' ); - let message = notification.QueryInterface(Ci.nsIPushObserverNotification); - equal(scope, 'https://example.com/page/1', 'Wrong scope'); - equal(message.pushEndpoint, 'https://example.org/updates/1', - 'Wrong push endpoint'); - strictEqual(message.version, 4, 'Wrong version'); + equal(notification, null, 'Unexpected data for Simple Push message'); yield waitForPromise(ackPromise, DEFAULT_TIMEOUT, 'Timed out waiting for string acknowledgement'); diff --git a/dom/push/test/xpcshell/test_quota_exceeded.js b/dom/push/test/xpcshell/test_quota_exceeded.js index b6c4b4b50512..383336579ec0 100644 --- a/dom/push/test/xpcshell/test_quota_exceeded.js +++ b/dom/push/test/xpcshell/test_quota_exceeded.js @@ -82,7 +82,7 @@ add_task(function* test_expiration_origin_threshold() { // different scopes, so each can send 5 notifications before we remove // their subscription. let updates = 0; - let notifyPromise = promiseObserverNotification('push-notification', (subject, data) => { + let notifyPromise = promiseObserverNotification('push-message', (subject, data) => { updates++; return updates == 6; }); diff --git a/dom/push/test/xpcshell/test_quota_with_notification.js b/dom/push/test/xpcshell/test_quota_with_notification.js index 23a7bb110aa8..4833a2a37fdc 100644 --- a/dom/push/test/xpcshell/test_quota_with_notification.js +++ b/dom/push/test/xpcshell/test_quota_with_notification.js @@ -55,7 +55,7 @@ add_task(function* test_expiration_origin_threshold() { let numMessages = 10; let updates = 0; - let notifyPromise = promiseObserverNotification('push-notification', (subject, data) => { + let notifyPromise = promiseObserverNotification('push-message', (subject, data) => { updates++; return updates == numMessages; }); diff --git a/dom/push/test/xpcshell/test_register_5xxCode_http2.js b/dom/push/test/xpcshell/test_register_5xxCode_http2.js index 77dc7fd4eadd..ab27205032a7 100644 --- a/dom/push/test/xpcshell/test_register_5xxCode_http2.js +++ b/dom/push/test/xpcshell/test_register_5xxCode_http2.js @@ -56,9 +56,6 @@ function run_test() { 'http2.retryInterval': 1000, 'http2.maxRetries': 2 }); - disableServiceWorkerEvents( - 'https://example.com/retry5xxCode' - ); run_next_test(); } diff --git a/dom/push/test/xpcshell/test_register_case.js b/dom/push/test/xpcshell/test_register_case.js index 69ce048a98c6..343192259c3e 100644 --- a/dom/push/test/xpcshell/test_register_case.js +++ b/dom/push/test/xpcshell/test_register_case.js @@ -10,9 +10,6 @@ const userAgentID = '1760b1f5-c3ba-40e3-9344-adef7c18ab12'; function run_test() { do_get_profile(); setPrefs(); - disableServiceWorkerEvents( - 'https://example.net/case' - ); run_next_test(); } diff --git a/dom/push/test/xpcshell/test_register_error_http2.js b/dom/push/test/xpcshell/test_register_error_http2.js index 3e363380efb4..f8f29225ee22 100644 --- a/dom/push/test/xpcshell/test_register_error_http2.js +++ b/dom/push/test/xpcshell/test_register_error_http2.js @@ -23,10 +23,6 @@ function run_test() { tlsProfile = prefs.getBoolPref("network.http.spdy.enforce-tls-profile"); - disableServiceWorkerEvents( - 'https://example.net/page/invalid-response' - ); - serverURL = "https://localhost:" + serverPort; run_next_test(); diff --git a/dom/push/test/xpcshell/test_register_flush.js b/dom/push/test/xpcshell/test_register_flush.js index c341a464ed2b..3b0e3d8d0043 100644 --- a/dom/push/test/xpcshell/test_register_flush.js +++ b/dom/push/test/xpcshell/test_register_flush.js @@ -15,10 +15,6 @@ function run_test() { requestTimeout: 1000, retryBaseInterval: 150 }); - disableServiceWorkerEvents( - 'https://example.com/page/1', - 'https://example.com/page/2' - ); run_next_test(); } @@ -36,7 +32,7 @@ add_task(function* test_register_flush() { }; yield db.put(record); - let notifyPromise = promiseObserverNotification('push-notification'); + let notifyPromise = promiseObserverNotification('push-message'); let ackDone; let ackPromise = new Promise(resolve => ackDone = after(2, resolve)); diff --git a/dom/push/test/xpcshell/test_register_invalid_channel.js b/dom/push/test/xpcshell/test_register_invalid_channel.js index a0d3c3e204a0..8d29d5fa9c2e 100644 --- a/dom/push/test/xpcshell/test_register_invalid_channel.js +++ b/dom/push/test/xpcshell/test_register_invalid_channel.js @@ -11,9 +11,6 @@ const channelID = 'cafed00d'; function run_test() { do_get_profile(); setPrefs(); - disableServiceWorkerEvents( - 'https://example.com/invalid-channel' - ); run_next_test(); } diff --git a/dom/push/test/xpcshell/test_register_invalid_endpoint.js b/dom/push/test/xpcshell/test_register_invalid_endpoint.js index 208d9320e5bc..ffafc64cdab0 100644 --- a/dom/push/test/xpcshell/test_register_invalid_endpoint.js +++ b/dom/push/test/xpcshell/test_register_invalid_endpoint.js @@ -11,9 +11,6 @@ const channelID = 'c0660af8-b532-4931-81f0-9fd27a12d6ab'; function run_test() { do_get_profile(); setPrefs(); - disableServiceWorkerEvents( - 'https://example.net/page/invalid-endpoint' - ); run_next_test(); } diff --git a/dom/push/test/xpcshell/test_register_invalid_json.js b/dom/push/test/xpcshell/test_register_invalid_json.js index adfa32928bb2..aa3b5c055bb9 100644 --- a/dom/push/test/xpcshell/test_register_invalid_json.js +++ b/dom/push/test/xpcshell/test_register_invalid_json.js @@ -14,9 +14,6 @@ function run_test() { requestTimeout: 1000, retryBaseInterval: 150 }); - disableServiceWorkerEvents( - 'https://example.net/page/invalid-json' - ); run_next_test(); } diff --git a/dom/push/test/xpcshell/test_register_no_id.js b/dom/push/test/xpcshell/test_register_no_id.js index 6fadfef30055..a562abf44731 100644 --- a/dom/push/test/xpcshell/test_register_no_id.js +++ b/dom/push/test/xpcshell/test_register_no_id.js @@ -15,9 +15,6 @@ function run_test() { requestTimeout: 1000, retryBaseInterval: 150 }); - disableServiceWorkerEvents( - 'https://example.com/incomplete' - ); run_next_test(); } diff --git a/dom/push/test/xpcshell/test_register_request_queue.js b/dom/push/test/xpcshell/test_register_request_queue.js index 47c430a32a0b..047846372bf9 100644 --- a/dom/push/test/xpcshell/test_register_request_queue.js +++ b/dom/push/test/xpcshell/test_register_request_queue.js @@ -11,9 +11,6 @@ function run_test() { requestTimeout: 1000, retryBaseInterval: 150 }); - disableServiceWorkerEvents( - 'https://example.com/page/1' - ); run_next_test(); } diff --git a/dom/push/test/xpcshell/test_register_rollback.js b/dom/push/test/xpcshell/test_register_rollback.js index aa7f49b2d1e0..35809a8189a2 100644 --- a/dom/push/test/xpcshell/test_register_rollback.js +++ b/dom/push/test/xpcshell/test_register_rollback.js @@ -15,9 +15,6 @@ function run_test() { requestTimeout: 1000, retryBaseInterval: 150 }); - disableServiceWorkerEvents( - 'https://example.com/storage-error' - ); run_next_test(); } diff --git a/dom/push/test/xpcshell/test_register_success.js b/dom/push/test/xpcshell/test_register_success.js index a0b52b54095e..68ead06637e8 100644 --- a/dom/push/test/xpcshell/test_register_success.js +++ b/dom/push/test/xpcshell/test_register_success.js @@ -15,9 +15,6 @@ function run_test() { requestTimeout: 1000, retryBaseInterval: 150 }); - disableServiceWorkerEvents( - 'https://example.org/1' - ); run_next_test(); } diff --git a/dom/push/test/xpcshell/test_register_success_http2.js b/dom/push/test/xpcshell/test_register_success_http2.js index a48c77793041..5217ebbfa3a9 100644 --- a/dom/push/test/xpcshell/test_register_success_http2.js +++ b/dom/push/test/xpcshell/test_register_success_http2.js @@ -37,11 +37,6 @@ function run_test() { serverURL = "https://localhost:" + serverPort; - disableServiceWorkerEvents( - 'https://example.org/1', - 'https://example.org/no_receiptEndpoint' - ); - run_next_test(); } diff --git a/dom/push/test/xpcshell/test_register_timeout.js b/dom/push/test/xpcshell/test_register_timeout.js index 05a46ecc837b..812077c447eb 100644 --- a/dom/push/test/xpcshell/test_register_timeout.js +++ b/dom/push/test/xpcshell/test_register_timeout.js @@ -14,9 +14,6 @@ function run_test() { requestTimeout: 1000, retryBaseInterval: 150 }); - disableServiceWorkerEvents( - 'https://example.net/page/timeout' - ); run_next_test(); } diff --git a/dom/push/test/xpcshell/test_register_wrong_id.js b/dom/push/test/xpcshell/test_register_wrong_id.js index 20bb5d4d396a..5b51adc4da9d 100644 --- a/dom/push/test/xpcshell/test_register_wrong_id.js +++ b/dom/push/test/xpcshell/test_register_wrong_id.js @@ -16,9 +16,6 @@ function run_test() { requestTimeout: 1000, retryBaseInterval: 150 }); - disableServiceWorkerEvents( - 'https://example.com/mismatched' - ); run_next_test(); } diff --git a/dom/push/test/xpcshell/test_register_wrong_type.js b/dom/push/test/xpcshell/test_register_wrong_type.js index 2b91d4e571c3..9ab06e849f51 100644 --- a/dom/push/test/xpcshell/test_register_wrong_type.js +++ b/dom/push/test/xpcshell/test_register_wrong_type.js @@ -13,9 +13,6 @@ function run_test() { requestTimeout: 1000, retryBaseInterval: 150 }); - disableServiceWorkerEvents( - 'https://example.com/mistyped' - ); run_next_test(); } diff --git a/dom/push/test/xpcshell/test_registration_success_http2.js b/dom/push/test/xpcshell/test_registration_success_http2.js index 7b3bdb133b19..e7d3c897f3ab 100644 --- a/dom/push/test/xpcshell/test_registration_success_http2.js +++ b/dom/push/test/xpcshell/test_registration_success_http2.js @@ -19,12 +19,6 @@ function run_test() { do_get_profile(); prefs = Cc["@mozilla.org/preferences-service;1"].getService(Ci.nsIPrefBranch); - disableServiceWorkerEvents( - 'https://example.net/a', - 'https://example.net/b', - 'https://example.net/c' - ); - run_next_test(); } diff --git a/dom/push/test/xpcshell/test_resubscribe_4xxCode_http2.js b/dom/push/test/xpcshell/test_resubscribe_4xxCode_http2.js index c8b43c3f4027..b3b8d2436659 100644 --- a/dom/push/test/xpcshell/test_resubscribe_4xxCode_http2.js +++ b/dom/push/test/xpcshell/test_resubscribe_4xxCode_http2.js @@ -48,9 +48,8 @@ function run_test() { do_get_profile(); - disableServiceWorkerEvents( - 'https://example.com/page' - ); + servicePrefs.set('testing.notifyWorkers', false); + servicePrefs.set('testing.notifyAllObservers', true); run_next_test(); } diff --git a/dom/push/test/xpcshell/test_resubscribe_5xxCode_http2.js b/dom/push/test/xpcshell/test_resubscribe_5xxCode_http2.js index 0f7739e2ffcf..b8372130a9d6 100644 --- a/dom/push/test/xpcshell/test_resubscribe_5xxCode_http2.js +++ b/dom/push/test/xpcshell/test_resubscribe_5xxCode_http2.js @@ -56,9 +56,6 @@ function run_test() { 'http2.retryInterval': 1000, 'http2.maxRetries': 2 }); - disableServiceWorkerEvents( - 'https://example.com/page' - ); run_next_test(); } diff --git a/dom/push/test/xpcshell/test_resubscribe_listening_for_msg_error_http2.js b/dom/push/test/xpcshell/test_resubscribe_listening_for_msg_error_http2.js index 7416862c5906..5044f6483f85 100644 --- a/dom/push/test/xpcshell/test_resubscribe_listening_for_msg_error_http2.js +++ b/dom/push/test/xpcshell/test_resubscribe_listening_for_msg_error_http2.js @@ -44,9 +44,6 @@ function run_test() { 'http2.retryInterval': 1000, 'http2.maxRetries': 2 }); - disableServiceWorkerEvents( - 'https://example.com/page' - ); run_next_test(); } diff --git a/dom/push/test/xpcshell/test_unregister_success_http2.js b/dom/push/test/xpcshell/test_unregister_success_http2.js index 5b3f6719df0b..b6c77ca3d6d0 100644 --- a/dom/push/test/xpcshell/test_unregister_success_http2.js +++ b/dom/push/test/xpcshell/test_unregister_success_http2.js @@ -34,10 +34,6 @@ function run_test() { prefs.setIntPref("network.http.speculative-parallel-limit", oldPref); - disableServiceWorkerEvents( - 'https://example.com/page/unregister-success' - ); - run_next_test(); } diff --git a/dom/push/test/xpcshell/test_updateRecordNoEncryptionKeys_http2.js b/dom/push/test/xpcshell/test_updateRecordNoEncryptionKeys_http2.js index 093c2335665b..db1196585e6d 100644 --- a/dom/push/test/xpcshell/test_updateRecordNoEncryptionKeys_http2.js +++ b/dom/push/test/xpcshell/test_updateRecordNoEncryptionKeys_http2.js @@ -32,9 +32,6 @@ function run_test() { 'http2.retryInterval': 1000, 'http2.maxRetries': 2 }); - disableServiceWorkerEvents( - 'https://example.com/page' - ); run_next_test(); } diff --git a/dom/push/test/xpcshell/test_updateRecordNoEncryptionKeys_ws.js b/dom/push/test/xpcshell/test_updateRecordNoEncryptionKeys_ws.js index 3a16d90e1b16..353cece05041 100644 --- a/dom/push/test/xpcshell/test_updateRecordNoEncryptionKeys_ws.js +++ b/dom/push/test/xpcshell/test_updateRecordNoEncryptionKeys_ws.js @@ -12,11 +12,6 @@ function run_test() { setPrefs({ userAgentID, }); - disableServiceWorkerEvents( - 'https://example.com/page/1', - 'https://example.com/page/2', - 'https://example.com/page/3' - ); run_next_test(); }