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

Unified Diff: content/shell/browser/layout_test/layout_test_content_browser_client.h

Issue 622793002: Group the different permission related methods in the content api. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase Created 6 years, 2 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/shell/browser/layout_test/layout_test_content_browser_client.h
diff --git a/content/shell/browser/layout_test/layout_test_content_browser_client.h b/content/shell/browser/layout_test/layout_test_content_browser_client.h
index 3049c06a9717fbac0fc631453720f9161972e7e6..d6cb61fc9968532f68544cdeae198f988db38a16 100644
--- a/content/shell/browser/layout_test/layout_test_content_browser_client.h
+++ b/content/shell/browser/layout_test/layout_test_content_browser_client.h
@@ -5,6 +5,7 @@
#ifndef CONTENT_SHELL_BROWSER_LAYOUT_TEST_LAYOUT_TEST_CONTENT_BROWSER_CLIENT_H_
#define CONTENT_SHELL_BROWSER_LAYOUT_TEST_LAYOUT_TEST_CONTENT_BROWSER_CLIENT_H_
+#include "content/public/browser/permission_type.h"
#include "content/shell/browser/shell_content_browser_client.h"
namespace content {
@@ -24,11 +25,13 @@ class LayoutTestContentBrowserClient : public ShellContentBrowserClient {
// ContentBrowserClient overrides.
void RenderProcessWillLaunch(RenderProcessHost* host) override;
- void RequestDesktopNotificationPermission(
- const GURL& source_origin,
- RenderFrameHost* render_frame_host,
- const base::Callback<void(blink::WebNotificationPermission)>& callback)
- override;
+ void RequestPermission(
+ PermissionType permission,
+ WebContents* web_contents,
+ int bridge_id,
+ const GURL& requesting_frame,
+ bool user_gesture,
+ const base::Callback<void(bool)>& result_callback) override;
blink::WebNotificationPermission CheckDesktopNotificationPermission(
const GURL& source_url,
ResourceContext* context,

Powered by Google App Engine
This is Rietveld 408576698