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

Unified Diff: chrome/browser/pref_service_unittest.cc

Issue 2823037: Add an ExtensionPrefStore, layered between the user prefs nad the managed pre... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 10 years, 5 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
« no previous file with comments | « chrome/browser/pref_service.cc ('k') | chrome/browser/pref_value_store.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/pref_service_unittest.cc
===================================================================
--- chrome/browser/pref_service_unittest.cc (revision 52073)
+++ chrome/browser/pref_service_unittest.cc (working copy)
@@ -64,7 +64,8 @@
// TODO(port): port this test to POSIX.
#if defined(OS_WIN)
TEST(PrefServiceTest, LocalizedPrefs) {
- PrefService prefs(new PrefValueStore(NULL, new DummyPrefStore(), NULL));
+ PrefService prefs(new PrefValueStore(NULL, NULL, new DummyPrefStore(),
+ NULL));
const wchar_t kBoolean[] = L"boolean";
const wchar_t kInteger[] = L"integer";
const wchar_t kString[] = L"string";
@@ -87,7 +88,8 @@
#endif
TEST(PrefServiceTest, NoObserverFire) {
- PrefService prefs(new PrefValueStore(NULL, new DummyPrefStore(), NULL));
+ PrefService prefs(new PrefValueStore(NULL, NULL, new DummyPrefStore(),
+ NULL));
const wchar_t pref_name[] = L"homepage";
prefs.RegisterStringPref(pref_name, "");
@@ -122,7 +124,8 @@
}
TEST(PrefServiceTest, HasPrefPath) {
- PrefService prefs(new PrefValueStore(NULL, new DummyPrefStore(), NULL));
+ PrefService prefs(new PrefValueStore(NULL, NULL, new DummyPrefStore(),
+ NULL));
const wchar_t path[] = L"fake.path";
@@ -146,7 +149,7 @@
dict->SetString(pref_name, std::string("http://www.cnn.com"));
DummyPrefStore* pref_store = new DummyPrefStore();
pref_store->set_prefs(dict);
- PrefService prefs(new PrefValueStore(NULL, pref_store, NULL));
+ PrefService prefs(new PrefValueStore(NULL, NULL, pref_store, NULL));
prefs.RegisterStringPref(pref_name, "");
const std::string new_pref_value("http://www.google.com/");
@@ -187,7 +190,7 @@
static const char value_[];
PrefServiceSetValueTest()
- : prefs_(new PrefValueStore(NULL, new DummyPrefStore(), NULL)),
+ : prefs_(new PrefValueStore(NULL, NULL, new DummyPrefStore(), NULL)),
name_string_(name_),
null_value_(Value::CreateNullValue()) {}
« no previous file with comments | « chrome/browser/pref_service.cc ('k') | chrome/browser/pref_value_store.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698