Chromium Code Reviews| Index: content/browser/push_messaging/push_messaging_message_filter.h |
| diff --git a/content/browser/push_messaging/push_messaging_message_filter.h b/content/browser/push_messaging/push_messaging_message_filter.h |
| index afd8d6743a47550fb68984286bbc2955cd54fe11..8087c0240d8324c7926315d153619a40275458ff 100644 |
| --- a/content/browser/push_messaging/push_messaging_message_filter.h |
| +++ b/content/browser/push_messaging/push_messaging_message_filter.h |
| @@ -24,7 +24,7 @@ class PushMessagingService; |
| class ServiceWorkerContextWrapper; |
| extern const char kPushSenderIdServiceWorkerKey[]; |
| -extern const char kPushRegistrationIdServiceWorkerKey[]; |
| +extern const char kPushSubscriptionIdServiceWorkerKey[]; |
| class PushMessagingMessageFilter : public BrowserMessageFilter { |
| public: |
| @@ -33,7 +33,7 @@ class PushMessagingMessageFilter : public BrowserMessageFilter { |
| ServiceWorkerContextWrapper* service_worker_context); |
| private: |
| - struct RegisterData; |
| + struct SubscribeData; |
| class Core; |
| friend class BrowserThread; |
| @@ -45,67 +45,67 @@ class PushMessagingMessageFilter : public BrowserMessageFilter { |
| void OnDestruct() const override; |
| bool OnMessageReceived(const IPC::Message& message) override; |
| - // Register methods on IO thread --------------------------------------------- |
| + // Subscribe methods on IO thread -------------------------------------------- |
| - void OnRegisterFromDocument(int render_frame_id, |
| - int request_id, |
| - const std::string& sender_id, |
| - bool user_visible, |
| - int64_t service_worker_registration_id); |
| + void OnSubscribeFromDocument(int render_frame_id, |
| + int request_id, |
| + const std::string& sender_id, |
| + bool user_visible, |
| + int64_t service_worker_registration_id); |
| - void OnRegisterFromWorker(int request_id, |
| - int64_t service_worker_registration_id, |
| - bool user_visible); |
| + void OnSubscribeFromWorker(int request_id, |
| + int64_t service_worker_registration_id, |
| + bool user_visible); |
| - void DidPersistSenderId(const RegisterData& data, |
| + void DidPersistSenderId(const SubscribeData& data, |
| const std::string& sender_id, |
| ServiceWorkerStatusCode service_worker_status); |
| // sender_id is ignored if data.FromDocument() is false. |
| - void CheckForExistingRegistration( |
| - const RegisterData& data, |
| + void CheckForExistingSubscription( |
| + const SubscribeData& data, |
| const std::string& sender_id); |
| // sender_id is ignored if data.FromDocument() is false. |
| - void DidCheckForExistingRegistration( |
| - const RegisterData& data, |
| + void DidCheckForExistingSubscription( |
| + const SubscribeData& data, |
| const std::string& sender_id, |
| - const std::string& push_registration_id, |
| + const std::string& push_subscription_id, |
| ServiceWorkerStatusCode service_worker_status); |
| - void DidGetSenderIdFromStorage(const RegisterData& data, |
| + void DidGetSenderIdFromStorage(const SubscribeData& data, |
| const std::string& sender_id, |
| ServiceWorkerStatusCode service_worker_status); |
| // Called via PostTask from UI thread. |
| - void PersistRegistrationOnIO(const RegisterData& data, |
| - const std::string& push_registration_id); |
| + void PersistSubscriptionOnIO(const SubscribeData& data, |
| + const std::string& push_subscribe_id); |
|
johnme
2015/05/07 13:31:25
Nit: "push_subscription_id"
|
| - void DidPersistRegistrationOnIO( |
| - const RegisterData& data, |
| - const std::string& push_registration_id, |
| + void DidPersistSubscriptionOnIO( |
| + const SubscribeData& data, |
| + const std::string& push_subscription_id, |
| ServiceWorkerStatusCode service_worker_status); |
| // Called both from IO thread, and via PostTask from UI thread. |
| - void SendRegisterError(const RegisterData& data, |
| - PushRegistrationStatus status); |
| + void SendSubscribeError(const SubscribeData& data, |
| + PushSubscriptionStatus status); |
| // Called both from IO thread, and via PostTask from UI thread. |
| - void SendRegisterSuccess(const RegisterData& data, |
| - PushRegistrationStatus status, |
| - const std::string& push_registration_id); |
| + void SendSubscribeSuccess(const SubscribeData& data, |
| + PushSubscriptionStatus status, |
| + const std::string& push_subscription_id); |
| - // Unregister methods on IO thread ------------------------------------------- |
| + // Unsubscribe methods on IO thread ------------------------------------------ |
| - void OnUnregister(int request_id, int64_t service_worker_registration_id); |
| + void OnUnsubscribe(int request_id, int64_t service_worker_registration_id); |
| - void UnregisterHavingGottenPushRegistrationId( |
| + void UnsubscribeHavingGottenPushSubscriptionId( |
| int request_id, |
| int64_t service_worker_registration_id, |
| const GURL& requesting_origin, |
| - const std::string& push_registration_id, |
| + const std::string& push_subscription_id, |
| ServiceWorkerStatusCode service_worker_status); |
| - void UnregisterHavingGottenSenderId( |
| + void UnsubscribeHavingGottenSenderId( |
| int request_id, |
| int64_t service_worker_registration_id, |
| const GURL& requesting_origin, |
| @@ -113,25 +113,25 @@ class PushMessagingMessageFilter : public BrowserMessageFilter { |
| ServiceWorkerStatusCode service_worker_status); |
| // Called via PostTask from UI thread. |
| - void ClearRegistrationData(int request_id, |
| + void ClearSubscriptionData(int request_id, |
| int64_t service_worker_registration_id, |
| - PushUnregistrationStatus unregistration_status); |
| + PushUnsubscriptionStatus unsubscription_status); |
| - void DidClearRegistrationData(int request_id, |
| - PushUnregistrationStatus unregistration_status, |
| + void DidClearSubscriptionData(int request_id, |
| + PushUnsubscriptionStatus unsubscription_status, |
| ServiceWorkerStatusCode service_worker_status); |
| // Called both from IO thread, and via PostTask from UI thread. |
| - void DidUnregister(int request_id, |
| - PushUnregistrationStatus unregistration_status); |
| + void DidUnsubscribe(int request_id, |
| + PushUnsubscriptionStatus unsubscribe_status); |
| - // GetRegistration methods on IO thread -------------------------------------- |
| + // GetSubscription methods on IO thread -------------------------------------- |
| - void OnGetRegistration(int request_id, |
| + void OnGetSubscription(int request_id, |
| int64_t service_worker_registration_id); |
| - void DidGetRegistration(int request_id, |
| - const std::string& push_registration_id, |
| + void DidGetSubscription(int request_id, |
| + const std::string& push_subscription_id, |
| ServiceWorkerStatusCode status); |
| // GetPermission methods on IO thread ---------------------------------------- |