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

Unified Diff: chrome/test/data/push_messaging/test.html

Issue 640593003: Remove the sender_id from the Push API code path. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git/+/master
Patch Set: add TODOs Created 6 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
« no previous file with comments | « chrome/test/data/push_messaging/manifest.json ('k') | content/renderer/push_messaging_dispatcher.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/test/data/push_messaging/test.html
diff --git a/chrome/test/data/push_messaging/test.html b/chrome/test/data/push_messaging/test.html
index 407695698f89695224c755778a9469741c1da770..1a3bd65213324bd34421982620227f3d9f061647 100644
--- a/chrome/test/data/push_messaging/test.html
+++ b/chrome/test/data/push_messaging/test.html
@@ -2,6 +2,7 @@
<html>
<head>
<title>Push API Test</title>
+ <link rel="manifest" href="manifest.json">
<script>
function sendResultToTest(result) {
console.log(result);
@@ -21,9 +22,10 @@
}, sendErrorToTest);
}
- function registerPush(senderId) {
+ function registerPush() {
navigator.serviceWorker.ready.then(function() {
- navigator.push.register(senderId).then(function(pushRegistration) {
+ // TODO(peter): Remove the argument once Blink no longer requires it.
+ navigator.push.register('DEPRECATED').then(function(pushRegistration) {
sendResultToTest(pushRegistration.pushEndpoint + ' - ' + pushRegistration.pushRegistrationId);
}, sendErrorToTest);
}, sendErrorToTest);
« no previous file with comments | « chrome/test/data/push_messaging/manifest.json ('k') | content/renderer/push_messaging_dispatcher.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698