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

Unified Diff: chrome/browser/prefs/pref_value_store.cc

Issue 5213002: Fix for Bug 50726 "Save extension list and "winning" prefs from extensions" (Closed) Base URL: http://git.chromium.org/git/chromium.git@trunk
Patch Set: Lint issues and factored out renaming of InMemoryPrefStore Created 10 years, 1 month 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/prefs/pref_value_store.cc
diff --git a/chrome/browser/prefs/pref_value_store.cc b/chrome/browser/prefs/pref_value_store.cc
index 84b6919ec7067db82354d49ea8596d67b7c1a504..7e96ea3d1f5b0038c8f0a6fb36be11a0fca4df01 100644
--- a/chrome/browser/prefs/pref_value_store.cc
+++ b/chrome/browser/prefs/pref_value_store.cc
@@ -5,7 +5,6 @@
#include "chrome/browser/prefs/pref_value_store.h"
#include "chrome/browser/browser_thread.h"
-#include "chrome/browser/extensions/extension_pref_store.h"
#include "chrome/browser/policy/configuration_policy_pref_store.h"
#include "chrome/browser/prefs/command_line_pref_store.h"
#include "chrome/browser/prefs/in_memory_pref_store.h"
@@ -41,7 +40,7 @@ PrefValueStore* PrefValueStore::CreatePrefValueStore(
using policy::ConfigurationPolicyPrefStore;
ConfigurationPolicyPrefStore* managed = NULL;
ConfigurationPolicyPrefStore* device_management = NULL;
- ExtensionPrefStore* extension = NULL;
+ InMemoryPrefStore* extension = NULL;
CommandLinePrefStore* command_line = NULL;
ConfigurationPolicyPrefStore* recommended = NULL;
@@ -56,7 +55,7 @@ PrefValueStore* PrefValueStore::CreatePrefValueStore(
device_management =
ConfigurationPolicyPrefStore::CreateDeviceManagementPolicyPrefStore(
profile);
- extension = new ExtensionPrefStore(profile, PrefNotifier::EXTENSION_STORE);
+ extension = new InMemoryPrefStore();
command_line = new CommandLinePrefStore(CommandLine::ForCurrentProcess());
recommended =
ConfigurationPolicyPrefStore::CreateRecommendedPolicyPrefStore();
@@ -154,7 +153,8 @@ bool PrefValueStore::PrefHasChanged(const char* path,
return true;
PrefNotifier::PrefStoreType controller = ControllingPrefStoreForPref(path);
- DCHECK(controller != PrefNotifier::INVALID_STORE);
+ DCHECK(controller != PrefNotifier::INVALID_STORE)
+ << "Invalid controller for path " << path;
if (controller == PrefNotifier::INVALID_STORE)
return true;
@@ -463,6 +463,10 @@ bool PrefValueStore::HasPolicyConflictingUserProxySettings() {
return false;
}
+PrefStore* PrefValueStore::GetExtensionPrefStore() {
+ return pref_stores_[PrefNotifier::EXTENSION_STORE].get();
Mattias Nissler (ping if slow) 2010/11/19 16:52:20 Fix indentation.
battre (please use the other) 2010/11/19 18:00:39 Done.
+}
+
PrefValueStore::PrefValueStore(PrefStore* managed_platform_prefs,
PrefStore* device_management_prefs,
PrefStore* extension_prefs,

Powered by Google App Engine
This is Rietveld 408576698