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

Unified Diff: chrome/browser/geolocation/chrome_geolocation_permission_context.cc

Issue 107413006: Dismiss EME infobar when WebMediaPlayer is destroyed. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: make FakeProfile class happy Created 6 years, 11 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/geolocation/chrome_geolocation_permission_context.cc
diff --git a/chrome/browser/geolocation/chrome_geolocation_permission_context.cc b/chrome/browser/geolocation/chrome_geolocation_permission_context.cc
index 6f6d9b66a080f221a417017d8995e6b026c683c0..616c1b14f310855465f79a52cc718e34dc20eaef 100644
--- a/chrome/browser/geolocation/chrome_geolocation_permission_context.cc
+++ b/chrome/browser/geolocation/chrome_geolocation_permission_context.cc
@@ -62,8 +62,7 @@ void ChromeGeolocationPermissionContext::RequestGeolocationPermission(
content::WebContents* web_contents =
tab_util::GetWebContentsByID(render_process_id, render_view_id);
- const PermissionRequestID id(render_process_id, render_view_id,
- bridge_id);
+ const PermissionRequestID id(render_process_id, render_view_id, bridge_id, 0);
ExtensionService* extension_service =
extensions::ExtensionSystem::Get(profile_)->extension_service();
if (extension_service) {
@@ -112,8 +111,8 @@ void ChromeGeolocationPermissionContext::CancelGeolocationPermissionRequest(
int render_view_id,
int bridge_id,
const GURL& requesting_frame) {
- CancelPendingInfoBarRequest(PermissionRequestID(
- render_process_id, render_view_id, bridge_id));
+ CancelPendingInfobarRequest(PermissionRequestID(
+ render_process_id, render_view_id, bridge_id, 0));
}
void ChromeGeolocationPermissionContext::DecidePermission(
@@ -193,13 +192,13 @@ PermissionQueueController*
CONTENT_SETTINGS_TYPE_GEOLOCATION);
}
-void ChromeGeolocationPermissionContext::CancelPendingInfoBarRequest(
+void ChromeGeolocationPermissionContext::CancelPendingInfobarRequest(
const PermissionRequestID& id) {
if (!content::BrowserThread::CurrentlyOn(content::BrowserThread::UI)) {
content::BrowserThread::PostTask(
content::BrowserThread::UI, FROM_HERE,
base::Bind(
- &ChromeGeolocationPermissionContext::CancelPendingInfoBarRequest,
+ &ChromeGeolocationPermissionContext::CancelPendingInfobarRequest,
this, id));
return;
}

Powered by Google App Engine
This is Rietveld 408576698