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

Unified Diff: content/child/push_messaging/push_dispatcher.cc

Issue 1134733006: Push API: use (un)subscription instead of (un)registration (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixed pre-submit warning 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: content/child/push_messaging/push_dispatcher.cc
diff --git a/content/child/push_messaging/push_dispatcher.cc b/content/child/push_messaging/push_dispatcher.cc
index 8cc33fc0dd20e53164d0fa60fbb94cd763af0b36..430344336cef9f1c473f79b0a8f99ca388c96185 100644
--- a/content/child/push_messaging/push_dispatcher.cc
+++ b/content/child/push_messaging/push_dispatcher.cc
@@ -27,14 +27,14 @@ bool PushDispatcher::ShouldHandleMessage(const IPC::Message& msg) const {
// of the API functionality requires a direct association with a document and
// a frame, and for those cases the IPC messages are handled by a
// RenderFrameObserver.
- return msg.type() == PushMessagingMsg_RegisterFromWorkerSuccess::ID ||
- msg.type() == PushMessagingMsg_RegisterFromWorkerError::ID ||
+ return msg.type() == PushMessagingMsg_SubscribeFromWorkerSuccess::ID ||
+ msg.type() == PushMessagingMsg_SubscribeFromWorkerError::ID ||
msg.type() == PushMessagingMsg_GetRegistrationSuccess::ID ||
msg.type() == PushMessagingMsg_GetRegistrationError::ID ||
msg.type() == PushMessagingMsg_GetPermissionStatusSuccess::ID ||
msg.type() == PushMessagingMsg_GetPermissionStatusError::ID ||
- msg.type() == PushMessagingMsg_UnregisterSuccess::ID ||
- msg.type() == PushMessagingMsg_UnregisterError::ID;
+ msg.type() == PushMessagingMsg_UnsubscribeSuccess::ID ||
+ msg.type() == PushMessagingMsg_UnsubscribeError::ID;
}
void PushDispatcher::OnFilteredMessageReceived(const IPC::Message& msg) {
« no previous file with comments | « content/browser/push_messaging/push_messaging_message_filter.cc ('k') | content/child/push_messaging/push_provider.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698