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

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

Issue 12079097: Introduce PrefRegistrySyncable, simplifying PrefServiceSyncable. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: One more merge to head. Created 7 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/extension_prefs_unittest.cc
diff --git a/chrome/browser/extensions/extension_prefs_unittest.cc b/chrome/browser/extensions/extension_prefs_unittest.cc
index d9553232df714fe00b6b40bdbae041148fdf0ed0..82b62d7c7bc73994a307f357c2d85bf62ae98143 100644
--- a/chrome/browser/extensions/extension_prefs_unittest.cc
+++ b/chrome/browser/extensions/extension_prefs_unittest.cc
@@ -15,6 +15,8 @@
#include "chrome/browser/extensions/extension_pref_value_map.h"
#include "chrome/browser/extensions/extension_prefs.h"
#include "chrome/browser/prefs/mock_pref_change_callback.h"
+#include "chrome/browser/prefs/pref_registry_syncable.h"
+#include "chrome/browser/prefs/pref_service_syncable.h"
#include "chrome/browser/prefs/scoped_user_pref_update.h"
#include "chrome/common/chrome_paths.h"
#include "chrome/common/extensions/extension_manifest_constants.h"
@@ -59,10 +61,10 @@ ExtensionPrefsTest::ExtensionPrefsTest()
ExtensionPrefsTest::~ExtensionPrefsTest() {
}
-void ExtensionPrefsTest::RegisterPreferences() {}
+void ExtensionPrefsTest::RegisterPreferences(PrefRegistrySyncable* registry) {}
void ExtensionPrefsTest::SetUp() {
- RegisterPreferences();
+ RegisterPreferences(prefs_.pref_registry());
Initialize();
}
@@ -70,8 +72,9 @@ void ExtensionPrefsTest::TearDown() {
Verify();
// Reset ExtensionPrefs, and re-verify.
+ prefs_.ResetPrefRegistry();
+ RegisterPreferences(prefs_.pref_registry());
prefs_.RecreateExtensionPrefs();
- RegisterPreferences();
Verify();
prefs_.pref_service()->CommitPendingWrite();
message_loop_.RunUntilIdle();
@@ -819,15 +822,16 @@ ExtensionPrefsPrepopulatedTest::ExtensionPrefsPrepopulatedTest()
ExtensionPrefsPrepopulatedTest::~ExtensionPrefsPrepopulatedTest() {}
-void ExtensionPrefsPrepopulatedTest::RegisterPreferences() {
- prefs()->pref_service()->RegisterStringPref(
- kPref1, kDefaultPref1, PrefServiceSyncable::UNSYNCABLE_PREF);
- prefs()->pref_service()->RegisterStringPref(
- kPref2, kDefaultPref2, PrefServiceSyncable::UNSYNCABLE_PREF);
- prefs()->pref_service()->RegisterStringPref(
- kPref3, kDefaultPref3, PrefServiceSyncable::UNSYNCABLE_PREF);
- prefs()->pref_service()->RegisterStringPref(
- kPref4, kDefaultPref4, PrefServiceSyncable::UNSYNCABLE_PREF);
+void ExtensionPrefsPrepopulatedTest::RegisterPreferences(
+ PrefRegistrySyncable* registry) {
+ registry->RegisterStringPref(
+ kPref1, kDefaultPref1, PrefRegistrySyncable::UNSYNCABLE_PREF);
+ registry->RegisterStringPref(
+ kPref2, kDefaultPref2, PrefRegistrySyncable::UNSYNCABLE_PREF);
+ registry->RegisterStringPref(
+ kPref3, kDefaultPref3, PrefRegistrySyncable::UNSYNCABLE_PREF);
+ registry->RegisterStringPref(
+ kPref4, kDefaultPref4, PrefRegistrySyncable::UNSYNCABLE_PREF);
}
void ExtensionPrefsPrepopulatedTest::InstallExtControlledPref(

Powered by Google App Engine
This is Rietveld 408576698