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

Unified Diff: chrome/browser/storage/durable_storage_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/storage/durable_storage_permission_context.cc
diff --git a/chrome/browser/storage/durable_storage_permission_context.cc b/chrome/browser/storage/durable_storage_permission_context.cc
index 5ea22bf0ac1d2e45cfda3fbd4580faa56c3396fc..81de0f8e47e42017bf9fc020c22efb7b9c31d409 100644
--- a/chrome/browser/storage/durable_storage_permission_context.cc
+++ b/chrome/browser/storage/durable_storage_permission_context.cc
@@ -49,8 +49,8 @@ void DurableStoragePermissionContext::DecidePermission(
// Durable is only allowed to be granted to the top-level origin. Embedding
// origin is the last committed navigation origin to the web contents.
if (requesting_origin != embedding_origin) {
- NotifyPermissionSet(id, requesting_origin, embedding_origin, callback,
- false /* persist */, CONTENT_SETTING_DEFAULT);
+ NotifyPermissionSet(web_contents, id, requesting_origin, embedding_origin,
+ callback, false /* persist */, CONTENT_SETTING_DEFAULT);
return;
}
@@ -59,8 +59,8 @@ void DurableStoragePermissionContext::DecidePermission(
CookieSettingsFactory::GetForProfile(profile());
if (!cookie_settings->IsSettingCookieAllowed(requesting_origin,
requesting_origin)) {
- NotifyPermissionSet(id, requesting_origin, embedding_origin, callback,
- false /* persist */, CONTENT_SETTING_DEFAULT);
+ NotifyPermissionSet(web_contents, id, requesting_origin, embedding_origin,
+ callback, false /* persist */, CONTENT_SETTING_DEFAULT);
return;
}
@@ -78,14 +78,14 @@ void DurableStoragePermissionContext::DecidePermission(
for (const auto& important_site : important_sites) {
if (important_site.registerable_domain == registerable_domain) {
- NotifyPermissionSet(id, requesting_origin, embedding_origin, callback,
- true /* persist */, CONTENT_SETTING_ALLOW);
+ NotifyPermissionSet(web_contents, id, requesting_origin, embedding_origin,
+ callback, true /* persist */, CONTENT_SETTING_ALLOW);
return;
}
}
- NotifyPermissionSet(id, requesting_origin, embedding_origin, callback,
- false /* persist */, CONTENT_SETTING_DEFAULT);
+ NotifyPermissionSet(web_contents, id, requesting_origin, embedding_origin,
+ callback, false /* persist */, CONTENT_SETTING_DEFAULT);
}
void DurableStoragePermissionContext::UpdateContentSetting(

Powered by Google App Engine
This is Rietveld 408576698