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

Unified Diff: chrome/browser/extensions/api/preference/preference_api.cc

Issue 184043024: Limit scope of settings API configuration and proxy permission (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Addressed Bernhard's comment Created 6 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/extensions/api/preference/preference_api.cc
diff --git a/chrome/browser/extensions/api/preference/preference_api.cc b/chrome/browser/extensions/api/preference/preference_api.cc
index 93f1c97ad1ebc40b998a60d5009c539d40001cd8..3c30a1dff53d665023767125973f6231e313affe 100644
--- a/chrome/browser/extensions/api/preference/preference_api.cc
+++ b/chrome/browser/extensions/api/preference/preference_api.cc
@@ -453,7 +453,9 @@ void PreferenceAPI::InitExtensionControlledPrefs(
extension_id != extension_ids.end(); ++extension_id) {
base::Time install_time = prefs->GetInstallTime(*extension_id);
bool is_enabled = !prefs->IsExtensionDisabled(*extension_id);
- value_map->RegisterExtension(*extension_id, install_time, is_enabled);
+ bool is_incognito_enabled = prefs->IsIncognitoEnabled(*extension_id);
not at google - send to devlin 2014/03/05 17:14:06 Perhaps you should be using util::IsIncognitoEnabl
battre 2014/03/05 17:38:43 Please take a look at the first version of this CL
+ value_map->RegisterExtension(
+ *extension_id, install_time, is_enabled, is_incognito_enabled);
prefs->content_settings_store()->RegisterExtension(
*extension_id, install_time, is_enabled);

Powered by Google App Engine
This is Rietveld 408576698