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

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

Issue 2627853002: Show the search geolocation disclosure from geolocation API use. (Closed)
Patch Set: Fix ChromeOS Created 3 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/geolocation_permission_context.cc
diff --git a/chrome/browser/geolocation/geolocation_permission_context.cc b/chrome/browser/geolocation/geolocation_permission_context.cc
index 38ab762bb4310d2391873e420a50fd1f735e9577..3172294063fe30e3d44df87123f00e7e14a9b0a4 100644
--- a/chrome/browser/geolocation/geolocation_permission_context.cc
+++ b/chrome/browser/geolocation/geolocation_permission_context.cc
@@ -41,12 +41,9 @@ void GeolocationPermissionContext::DecidePermission(
if (permission_set) {
ContentSetting content_setting =
new_permission ? CONTENT_SETTING_ALLOW : CONTENT_SETTING_BLOCK;
- NotifyPermissionSet(id,
- requesting_origin,
+ NotifyPermissionSet(web_contents, id, requesting_origin,
web_contents->GetLastCommittedURL().GetOrigin(),
- callback,
- false /* persist */,
- content_setting);
+ callback, false /* persist */, content_setting);
}
return;
}

Powered by Google App Engine
This is Rietveld 408576698