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

Unified Diff: chrome/browser/extensions/component_loader_unittest.cc

Issue 11570009: Split PrefService into PrefService, PrefServiceSimple and PrefServiceSyncable. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Merge to head again, previous had unrelated broken win_rel test. Created 8 years 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
« no previous file with comments | « chrome/browser/extensions/component_loader.cc ('k') | chrome/browser/extensions/default_apps.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/extensions/component_loader_unittest.cc
diff --git a/chrome/browser/extensions/component_loader_unittest.cc b/chrome/browser/extensions/component_loader_unittest.cc
index fa64fd4786131eaa63481d40180e5112e09d9442..288a16fdc16a17099f427886c818d88bc893d9a3 100644
--- a/chrome/browser/extensions/component_loader_unittest.cc
+++ b/chrome/browser/extensions/component_loader_unittest.cc
@@ -75,7 +75,7 @@ class ComponentLoaderTest : public testing::Test {
ComponentLoaderTest() :
// Note: we pass the same pref service here, to stand in for both
// user prefs and local state.
- component_loader_(&extension_service_, &prefs_, &prefs_) {
+ component_loader_(&extension_service_, &prefs_, &local_state_) {
}
void SetUp() {
@@ -91,18 +91,23 @@ class ComponentLoaderTest : public testing::Test {
&manifest_contents_));
// Register the user prefs that ComponentLoader will read.
- prefs_.RegisterStringPref(prefs::kEnterpriseWebStoreURL, std::string());
- prefs_.RegisterStringPref(prefs::kEnterpriseWebStoreName, std::string());
+ prefs_.RegisterStringPref(prefs::kEnterpriseWebStoreURL,
+ std::string(),
+ PrefServiceSyncable::UNSYNCABLE_PREF);
+ prefs_.RegisterStringPref(prefs::kEnterpriseWebStoreName,
+ std::string(),
+ PrefServiceSyncable::UNSYNCABLE_PREF);
// Register the local state prefs.
#if defined(OS_CHROMEOS)
- prefs_.RegisterBooleanPref(prefs::kSpokenFeedbackEnabled, false);
+ local_state_.RegisterBooleanPref(prefs::kSpokenFeedbackEnabled, false);
#endif
}
protected:
MockExtensionService extension_service_;
- TestingPrefService prefs_;
+ TestingPrefServiceSyncable prefs_;
+ TestingPrefServiceSimple local_state_;
ComponentLoader component_loader_;
// The root directory of the text extension.
« no previous file with comments | « chrome/browser/extensions/component_loader.cc ('k') | chrome/browser/extensions/default_apps.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698