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

Unified Diff: chrome/browser/permissions/permission_manager.h

Issue 2713083003: Use ContentSetting in chrome/ instead of PermissionStatus (Closed)
Patch Set: really fix build? Created 3 years, 10 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: chrome/browser/permissions/permission_manager.h
diff --git a/chrome/browser/permissions/permission_manager.h b/chrome/browser/permissions/permission_manager.h
index b8be948bbb651345548974a21aaf4cea2f517abc..aa48bf918f82b360dc9de603aad6198aae8df9f3 100644
--- a/chrome/browser/permissions/permission_manager.h
+++ b/chrome/browser/permissions/permission_manager.h
@@ -38,24 +38,21 @@ class PermissionManager : public KeyedService,
// are for the content::PermissionManager overrides and shouldn't be used
// from chrome/.
- int RequestPermission(
- ContentSettingsType permission,
- content::RenderFrameHost* render_frame_host,
- const GURL& requesting_origin,
- bool user_gesture,
- const base::Callback<void(blink::mojom::PermissionStatus)>& callback);
+ int RequestPermission(ContentSettingsType permission,
+ content::RenderFrameHost* render_frame_host,
+ const GURL& requesting_origin,
+ bool user_gesture,
+ const base::Callback<void(ContentSetting)>& callback);
int RequestPermissions(
const std::vector<ContentSettingsType>& permissions,
content::RenderFrameHost* render_frame_host,
const GURL& requesting_origin,
bool user_gesture,
- const base::Callback<
- void(const std::vector<blink::mojom::PermissionStatus>&)>& callback);
+ const base::Callback<void(const std::vector<ContentSetting>&)>& callback);
- blink::mojom::PermissionStatus GetPermissionStatus(
- ContentSettingsType permission,
- const GURL& requesting_origin,
- const GURL& embedding_origin);
+ ContentSetting GetPermissionStatus(ContentSettingsType permission,
+ const GURL& requesting_origin,
+ const GURL& embedding_origin);
// content::PermissionManager implementation.
int RequestPermission(
@@ -111,10 +108,9 @@ class PermissionManager : public KeyedService,
// one permission, it will wait for the remaining permissions to be resolved.
// When all the permissions have been resolved, the PendingRequest's callback
// is run.
- void OnPermissionsRequestResponseStatus(
- int request_id,
- int permission_id,
- blink::mojom::PermissionStatus status);
+ void OnPermissionsRequestResponseStatus(int request_id,
+ int permission_id,
+ ContentSetting status);
// content_settings::Observer implementation.
void OnContentSettingChanged(const ContentSettingsPattern& primary_pattern,

Powered by Google App Engine
This is Rietveld 408576698