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

Unified Diff: content/browser/frame_host/render_frame_host_impl.h

Issue 381633005: Refactor Web Notification permission requests. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 6 years, 5 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/browser/frame_host/render_frame_host_impl.h
diff --git a/content/browser/frame_host/render_frame_host_impl.h b/content/browser/frame_host/render_frame_host_impl.h
index 2d7036d0b1a8ae4297c36dca318cf86318daf62d..2b030fb97f653359e55619424baaa808e98f8739 100644
--- a/content/browser/frame_host/render_frame_host_impl.h
+++ b/content/browser/frame_host/render_frame_host_impl.h
@@ -18,6 +18,7 @@
#include "content/public/browser/render_frame_host.h"
#include "content/public/common/javascript_message_type.h"
#include "content/public/common/page_transition_types.h"
+#include "third_party/WebKit/public/platform/WebNotificationPermission.h"
#include "third_party/WebKit/public/web/WebTextDirection.h"
class GURL;
@@ -231,6 +232,8 @@ class CONTENT_EXPORT RenderFrameHostImpl : public RenderFrameHost {
const base::string16& message,
bool is_reload,
IPC::Message* reply_msg);
+ void OnRequestPlatformNotificationPermission(const GURL& origin,
+ int request_id);
void OnRequestDesktopNotificationPermission(const GURL& origin,
int callback_id);
void OnShowDesktopNotification(
@@ -252,6 +255,8 @@ class CONTENT_EXPORT RenderFrameHostImpl : public RenderFrameHost {
// it will be used to kill processes that commit unauthorized URLs.
bool CanCommitURL(const GURL& url);
+ void PlatformNotificationPermissionRequestDone(const GURL& origin,
+ int request_id);
void DesktopNotificationPermissionRequestDone(int callback_context);
// For now, RenderFrameHosts indirectly keep RenderViewHosts alive via a

Powered by Google App Engine
This is Rietveld 408576698