Blink implementation of PushManager#hasPermission()

public/platform/WebPushPermissionCallback.h

Index: public/platform/WebPushPermissionCallback.h
diff --git a/public/platform/WebPushPermissionCallback.h b/public/platform/WebPushPermissionCallback.h
new file mode 100644
index 0000000000000000000000000000000000000000..0f777854ce241c0c13d872b6bcfb5dc27810d7de
--- /dev/null
+++ b/public/platform/WebPushPermissionCallback.h
@@ -0,0 +1,26 @@
+// 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 WebPushPermissionCallback_h
+#define WebPushPermissionCallback_h
+
+#include "WebPushPermissionStatus.h"
+
+// Interface to be used by the embedder in order
+// to inform Blink when once the permission status for push has been checked.
+namespace blink {
+
+class WebPushPermissionCallback {

[Peter Beverloo, 2014/10/23 11:33:37]

You can replace this entire class by including public/platform/WebCallbacks.h
and doing:

typedef WebCallbacks<WebPushPermissionStatus, void> WebPushPermissionCallback;

(You may have to add a specialization of WebCallbacks<Type, void>.)

[Miguel Garcia, 2014/10/23 13:59:52]

As discussed offline, the template expands to onSuccess(S*) which for an S of
type enum seems like an overkill to pass a pointer. I am therefore leaving it as
is.

+public:
+    virtual ~WebPushPermissionCallback() { }
+
+    // Called if the embedder is able to check the status of the push permission.
+    virtual void onSuccess(WebPushPermissionStatus) = 0;
+
+    // Called if for some reason the status of the push permission cannot be checked.
+    virtual void onError() = 0;
+};
+
+} // namespace blink
+#endif // WebPushPermissionCallback_h