Convert push_messaging IPC msgs into mojo interfaces

content/child/push_messaging/push_provider.h

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CONTENT_CHILD_PUSH_MESSAGING_PUSH_PROVIDER_H_
 #define CONTENT_CHILD_PUSH_MESSAGING_PUSH_PROVIDER_H_
 
 #include <stdint.h>
 
 #include <memory>
 #include <string>
 #include <vector>
 
-#include "base/id_map.h"
 #include "base/macros.h"
 #include "base/memory/ref_counted.h"
-#include "content/child/push_messaging/push_dispatcher.h"
+#include "content/common/push_messaging.mojom.h"
 #include "content/public/child/worker_thread.h"
 #include "content/public/common/push_messaging_status.h"
 #include "third_party/WebKit/public/platform/modules/push_messaging/WebPushError.h"
 #include "third_party/WebKit/public/platform/modules/push_messaging/WebPushProvider.h"
 
 class GURL;
 
 namespace blink {
 struct WebPushSubscriptionOptions;
 }
 
 namespace content {
 
-class ThreadSafeSender;
 struct PushSubscriptionOptions;
 
 blink::WebPushError PushRegistrationStatusToWebPushError(
     PushRegistrationStatus status);
 
 class PushProvider : public blink::WebPushProvider,
                      public WorkerThread::Observer {
  public:
   ~PushProvider() override;
 
-  // The |thread_safe_sender| and |push_dispatcher| are used if calling this
-  // leads to construction.
   static PushProvider* ThreadSpecificInstance(
-      ThreadSafeSender* thread_safe_sender,
-      PushDispatcher* push_dispatcher);
+      scoped_refptr<base::SingleThreadTaskRunner>& main_thread_task_runner);

[Peter Beverloo, 2017/02/16 16:08:33]

nit: const&

[ke.he, 2017/02/17 08:22:33]

Done.

+
+  static void BindRequest(mojom::PushMessagingRequest request);

[Peter Beverloo, 2017/02/16 16:08:32]

nit: this can be private

[ke.he, 2017/02/17 08:22:38]

Done.

 
   // WorkerThread::Observer implementation.
   void WillStopCurrentWorkerThread() override;
 
   // blink::WebPushProvider implementation.
   void subscribe(
       blink::WebServiceWorkerRegistration* service_worker_registration,
       const blink::WebPushSubscriptionOptions& options,
       std::unique_ptr<blink::WebPushSubscriptionCallbacks> callbacks) override;
   void unsubscribe(
       blink::WebServiceWorkerRegistration* service_worker_registration,
       std::unique_ptr<blink::WebPushUnsubscribeCallbacks> callbacks) override;
   void getSubscription(
       blink::WebServiceWorkerRegistration* service_worker_registration,
       std::unique_ptr<blink::WebPushSubscriptionCallbacks> callbacks) override;
   void getPermissionStatus(
       blink::WebServiceWorkerRegistration* service_worker_registration,
       const blink::WebPushSubscriptionOptions& options,
       std::unique_ptr<blink::WebPushPermissionStatusCallbacks> callbacks)
       override;
 
-  // Called by the PushDispatcher.
-  bool OnMessageReceived(const IPC::Message& message);
+ private:
+  explicit PushProvider(
+      scoped_refptr<base::SingleThreadTaskRunner>& main_thread_task_runner);

[Peter Beverloo, 2017/02/16 16:08:33]

nit: const&

[ke.he, 2017/02/17 08:22:34]

Done.

 
- private:
-  PushProvider(ThreadSafeSender* thread_safe_sender,
-               PushDispatcher* push_dispatcher);
+  void SubscribeCallback(

[Peter Beverloo, 2017/02/16 16:08:33]

nit: I have a mild preference of naming these DidFoo as opposed to FooCallback.
It conveys information about the fact that something happened. (Even if it
failed.) Up to you.

[ke.he, 2017/02/17 08:22:37]

Yeah, I'll rename it in later. Record this in 693366.

+      std::unique_ptr<blink::WebPushSubscriptionCallbacks> callbacks,
+      content::PushRegistrationStatus status,
+      const base::Optional<GURL>& endpoint,
+      const base::Optional<content::PushSubscriptionOptions>& options,
+      const base::Optional<std::vector<uint8_t>>& p256dh,
+      const base::Optional<std::vector<uint8_t>>& auth);
 
-  // IPC message handlers.
-  void OnSubscribeFromWorkerSuccess(int request_id,
-                                    const GURL& endpoint,
-                                    const PushSubscriptionOptions& options,
-                                    const std::vector<uint8_t>& p256dh,
-                                    const std::vector<uint8_t>& auth);
-  void OnSubscribeFromWorkerError(int request_id,
-                                  PushRegistrationStatus status);
-  void OnUnsubscribeSuccess(int request_id, bool did_unsubscribe);
-  void OnUnsubscribeError(int request_id,
-                          blink::WebPushError::ErrorType error_type,
-                          const std::string& error_message);
-  void OnGetSubscriptionSuccess(int request_id,
-                                const GURL& endpoint,
-                                const PushSubscriptionOptions& options,
-                                const std::vector<uint8_t>& p256dh,
-                                const std::vector<uint8_t>& auth);
-  void OnGetSubscriptionError(int request_id, PushGetRegistrationStatus status);
-  void OnGetPermissionStatusSuccess(int request_id,
-                                    blink::WebPushPermissionStatus status);
-  void OnGetPermissionStatusError(int request_id,
-                                  blink::WebPushError::ErrorType error);
+  void UnsubscribeCallback(
+      std::unique_ptr<blink::WebPushUnsubscribeCallbacks> callbacks,
+      bool is_success,
+      bool did_unsubscribe,
+      blink::WebPushError::ErrorType error_type,
+      const base::Optional<std::string>& error_message);
 
-  scoped_refptr<ThreadSafeSender> thread_safe_sender_;
-  scoped_refptr<PushDispatcher> push_dispatcher_;
+  void GetSubscriptionCallback(
+      std::unique_ptr<blink::WebPushSubscriptionCallbacks> callbacks,
+      content::PushGetRegistrationStatus status,
+      const base::Optional<GURL>& endpoint,
+      const base::Optional<content::PushSubscriptionOptions>& options,
+      const base::Optional<std::vector<uint8_t>>& p256dh,
+      const base::Optional<std::vector<uint8_t>>& auth);

[Peter Beverloo, 2017/02/16 16:08:33]

Also as a potential follow-up, we can extract {endpoint, options, p256dh, auth}
in a separate Mojo structure to avoid these four optional arguments set both for
subscribing and getting the active subscription.

[ke.he, 2017/02/17 08:22:34]

tracked by 693366.

 
-  // Stores the subscription callbacks with their request ids. This class owns
-  // the callbacks.
-  IDMap<std::unique_ptr<blink::WebPushSubscriptionCallbacks>>
-      subscription_callbacks_;
+  void GetPermissionStatusCallback(
+      std::unique_ptr<blink::WebPushPermissionStatusCallbacks> callbacks,
+      bool is_success,
+      blink::WebPushPermissionStatus status,
+      blink::WebPushError::ErrorType error);
 
-  // Stores the permission status callbacks with their request ids. This class
-  // owns the callbacks.
-  IDMap<std::unique_ptr<blink::WebPushPermissionStatusCallbacks>>
-      permission_status_callbacks_;
-
-  // Stores the unsubscription callbacks with their request ids. This class owns
-  // the callbacks.
-  IDMap<std::unique_ptr<blink::WebPushUnsubscribeCallbacks>>
-      unsubscribe_callbacks_;
+  mojom::PushMessagingPtr push_messaging_;

[Peter Beverloo, 2017/02/16 16:08:33]

nit: push_messaging_manager_

[ke.he, 2017/02/17 08:22:35]

Done.

 
   DISALLOW_COPY_AND_ASSIGN(PushProvider);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_CHILD_PUSH_MESSAGING_PUSH_PROVIDER_H_