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

Unified Diff: LayoutTests/http/tests/notifications/resources/click-forward-service-worker.js

Issue 789643003: The Service Worker notificationclick event should carry a notification. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years 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/notifications/resources/click-forward-service-worker.js
diff --git a/LayoutTests/http/tests/notifications/resources/click-forward-service-worker.js b/LayoutTests/http/tests/notifications/resources/click-forward-service-worker.js
index 0904d2db9acee06cda52129b5571c380dba3e074..39b128458e1f1eba47a13c8fa1bb4981b2500866 100644
--- a/LayoutTests/http/tests/notifications/resources/click-forward-service-worker.js
+++ b/LayoutTests/http/tests/notifications/resources/click-forward-service-worker.js
@@ -1,4 +1,4 @@
-// Sends a message to all clients when the notificationclick event gets received.
+// Sends a message when the notificationclick event gets received.
var messagePort = null;
addEventListener('message', function(event) {
@@ -7,6 +7,5 @@ addEventListener('message', function(event) {
});
addEventListener('notificationclick', function(event) {
- // FIXME: Include the notification's title in the confirmation reply.
- messagePort.postMessage('The notification has been clicked on.');
+ messagePort.postMessage('Clicked on Notification: ' + event.notification.title);
});

Powered by Google App Engine
This is Rietveld 408576698