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

Unified Diff: chrome/browser/content_settings/content_settings_policy_provider.cc

Issue 7088017: Split incognito extension content settings in session-only and persistent. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: review Created 9 years, 7 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/content_settings/content_settings_policy_provider.cc
diff --git a/chrome/browser/content_settings/content_settings_policy_provider.cc b/chrome/browser/content_settings/content_settings_policy_provider.cc
index 484d53648f4e8c2d0d427168f77c8a2525a7756d..55b43609184b7eb7a60bd5a51e631efdf00aac7e 100644
--- a/chrome/browser/content_settings/content_settings_policy_provider.cc
+++ b/chrome/browser/content_settings/content_settings_policy_provider.cc
@@ -178,8 +178,10 @@ void PolicyDefaultProvider::Observe(NotificationType type,
}
if (!is_off_the_record_) {
- NotifyObservers(ContentSettingsDetails(
- ContentSettingsPattern(), CONTENT_SETTINGS_TYPE_DEFAULT, ""));
+ ContentSettingsDetails details(ContentSettingsPattern(),
+ CONTENT_SETTINGS_TYPE_DEFAULT,
+ std::string());
+ NotifyObservers(details);
}
} else if (type == NotificationType::PROFILE_DESTROYED) {
DCHECK_EQ(profile_, Source<Profile>(source).ptr());
@@ -448,8 +450,10 @@ void PolicyProvider::Observe(NotificationType type,
*name == prefs::kManagedPopupsAllowedForUrls ||
*name == prefs::kManagedPopupsBlockedForUrls) {
ReadManagedContentSettings(true);
- NotifyObservers(ContentSettingsDetails(
- ContentSettingsPattern(), CONTENT_SETTINGS_TYPE_DEFAULT, ""));
+ ContentSettingsDetails details(ContentSettingsPattern(),
+ CONTENT_SETTINGS_TYPE_DEFAULT,
+ std::string());
+ NotifyObservers(details);
}
} else if (type == NotificationType::PROFILE_DESTROYED) {
DCHECK_EQ(profile_, Source<Profile>(source).ptr());

Powered by Google App Engine
This is Rietveld 408576698