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

Unified Diff: LayoutTests/http/tests/push_messaging/resources/instrumentation-service-worker.js

Issue 1148763005: Remove support for PushSubscription.subscriptionId. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 5 years, 7 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: LayoutTests/http/tests/push_messaging/resources/instrumentation-service-worker.js
diff --git a/LayoutTests/http/tests/push_messaging/resources/instrumentation-service-worker.js b/LayoutTests/http/tests/push_messaging/resources/instrumentation-service-worker.js
index 81634ca362c48ffe81ef904cfa0dc86433eb39fc..f635b2c97eda8cf6f1098a29576ee3983bc0950b 100644
--- a/LayoutTests/http/tests/push_messaging/resources/instrumentation-service-worker.js
+++ b/LayoutTests/http/tests/push_messaging/resources/instrumentation-service-worker.js
@@ -28,7 +28,6 @@ self.addEventListener('message', function(workerEvent) {
lastSeenSubscription = subscription;
port.postMessage({ command: event.data.command,
success: true,
- subscriptionId: subscription.subscriptionId,
endpoint: subscription.endpoint });
}).catch(makeErrorHandler(event.data.command));
break;
@@ -36,10 +35,10 @@ self.addEventListener('message', function(workerEvent) {
case 'getSubscription':
self.registration.pushManager.getSubscription().then(function(subscription) {
lastSeenSubscription = subscription;
- var subscriptionId = subscription ? subscription.subscriptionId : null;
+ var endpoint = subscription ? subscription.endpoint : null;
port.postMessage({ command: event.data.command,
success: true,
- subscriptionId: subscriptionId });
+ endpoint: endpoint });
}).catch(makeErrorHandler(event.data.command));
break;

Powered by Google App Engine
This is Rietveld 408576698