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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: content/public/browser/content_browser_client.h

Issue 459953002: Migrate geolocation permissions to the new common permission class. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 4 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
« chrome/browser/content_settings/permission_context_base.cc ('K') | « content/browser/geolocation/geolocation_dispatcher_host.cc ('k') | content/public/browser/content_browser_client.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/public/browser/content_browser_client.h
diff --git a/content/public/browser/content_browser_client.h b/content/public/browser/content_browser_client.h
index 23ea93765f8faa3b1750b26d06aaad0da2533a23..630a8c194b069a006204f7608abebf392b4adf90 100644
--- a/content/public/browser/content_browser_client.h
+++ b/content/public/browser/content_browser_client.h
@@ -450,8 +450,12 @@ class CONTENT_EXPORT ContentBrowserClient {
int bridge_id,
const GURL& requesting_frame,
bool user_gesture,
- base::Callback<void(bool)> result_callback,
- base::Closure* cancel_callback);
+ const base::Callback<void(bool)>& result_callback);
+
+ virtual void CancelGeolocationPermissionRequest(
+ WebContents* web_contents,
+ int bridge_id,
+ const GURL& requesting_frame);
// Invoked when the Geolocation API uses its permission.
virtual void DidUseGeolocationPermission(WebContents* web_contents,
« chrome/browser/content_settings/permission_context_base.cc ('K') | « content/browser/geolocation/geolocation_dispatcher_host.cc ('k') | content/public/browser/content_browser_client.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698