Implement PushManager#hasPermission

content/renderer/push_messaging_dispatcher.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_RENDERER_PUSH_MESSAGING_DISPATCHER_H_
 #define CONTENT_RENDERER_PUSH_MESSAGING_DISPATCHER_H_
 
 #include <string>
 
 #include "base/id_map.h"
 #include "content/public/common/push_messaging_status.h"
 #include "content/public/renderer/render_frame_observer.h"
 #include "third_party/WebKit/public/platform/WebPushClient.h"
+#include "third_party/WebKit/public/platform/WebPushPermissionCallback.h"
 
 class GURL;
 
 namespace IPC {
 class Message;
 }  // namespace IPC
 
 namespace blink {
+class WebPushPermissionCallback;

[Michael van Ouwerkerk, 2014/10/20 16:30:08]

No need as you already included it.

[Miguel Garcia, 2014/10/22 16:35:27]

Done (as in I don't include it anymore)

 class WebServiceWorkerProvider;
 class WebString;
 }  // namespace blink
 
 namespace content {
 
 struct Manifest;
 
 class PushMessagingDispatcher : public RenderFrameObserver,
                                 public blink::WebPushClient {
  public:
   explicit PushMessagingDispatcher(RenderFrame* render_frame);
   virtual ~PushMessagingDispatcher();
 
  private:
   // RenderFrame::Observer implementation.
   virtual bool OnMessageReceived(const IPC::Message& message) override;
 
   // WebPushClient implementation.
   virtual void registerPushMessaging(
       const blink::WebString& sender_id,
       blink::WebPushRegistrationCallbacks* callbacks,
-      blink::WebServiceWorkerProvider* service_worker_provider);
+      blink::WebServiceWorkerProvider* service_worker_provider) /* override */;

[Michael van Ouwerkerk, 2014/10/20 16:30:08]

Not sure what the common form is for indicating override in a comment, but this
form shows 0 hits in code search. Isn't there any other precedent?

[Miguel Garcia, 2014/10/22 16:35:27]

So I would prefer to leave it uncommnented but that will make the multi repo
changes hard while developing the spec. I rather leave it as is (instead of
removing it completetly) while the api stabilizes so we know they are overrides,
then at some point remove the comment.

+
+  virtual void permissionStatus(
+      blink::WebPushPermissionCallback* callback,
+      blink::WebServiceWorkerProvider* service_worker_provider) /* override */;
 
   void DoRegister(const std::string& sender_id,
                   blink::WebPushRegistrationCallbacks* callbacks,
                   blink::WebServiceWorkerProvider* service_worker_provider,
                   const Manifest& manifest);
 
   void OnRegisterSuccess(int32 callbacks_id,
                          const GURL& endpoint,
                          const std::string& registration_id);
 
   void OnRegisterError(int32 callbacks_id, PushMessagingStatus status);
 
+  void OnPermissionStatus(
+      int32 callback_id,
+      blink::WebPushPermissionCallback::PushPermissionStatus);
+
+
   IDMap<blink::WebPushRegistrationCallbacks, IDMapOwnPointer>
       registration_callbacks_;
+  IDMap<blink::WebPushPermissionCallback, IDMapOwnPointer>
+      permission_check_callbacks_;
+
 
   DISALLOW_COPY_AND_ASSIGN(PushMessagingDispatcher);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_RENDERER_PUSH_MESSAGING_DISPATCHER_H_