| Index: chrome/browser/prefs/pref_member_unittest.cc
|
| diff --git a/chrome/browser/prefs/pref_member_unittest.cc b/chrome/browser/prefs/pref_member_unittest.cc
|
| index 986fc69df877a8a9b6fb576adc510d4970aa25ac..2dbbac6e831de0be903735a45bd0660efa034bac 100644
|
| --- a/chrome/browser/prefs/pref_member_unittest.cc
|
| +++ b/chrome/browser/prefs/pref_member_unittest.cc
|
| @@ -51,103 +51,105 @@ class PrefMemberTestClass : public NotificationObserver {
|
| } // anonymous namespace
|
|
|
| TEST(PrefMemberTest, BasicGetAndSet) {
|
| - TestingPrefService prefs;
|
| - RegisterTestPrefs(&prefs);
|
| + scoped_ptr<TestingPrefService> prefs(
|
| + TestingPrefService::CreateTestingPrefService());
|
| + RegisterTestPrefs(prefs.get());
|
|
|
| // Test bool
|
| BooleanPrefMember boolean;
|
| - boolean.Init(kBoolPref, &prefs, NULL);
|
| + boolean.Init(kBoolPref, prefs.get(), NULL);
|
|
|
| // Check the defaults
|
| - EXPECT_FALSE(prefs.GetBoolean(kBoolPref));
|
| + EXPECT_FALSE(prefs->GetBoolean(kBoolPref));
|
| EXPECT_FALSE(boolean.GetValue());
|
| EXPECT_FALSE(*boolean);
|
|
|
| // Try changing through the member variable.
|
| boolean.SetValue(true);
|
| EXPECT_TRUE(boolean.GetValue());
|
| - EXPECT_TRUE(prefs.GetBoolean(kBoolPref));
|
| + EXPECT_TRUE(prefs->GetBoolean(kBoolPref));
|
| EXPECT_TRUE(*boolean);
|
|
|
| // Try changing back through the pref.
|
| - prefs.SetBoolean(kBoolPref, false);
|
| - EXPECT_FALSE(prefs.GetBoolean(kBoolPref));
|
| + prefs->SetBoolean(kBoolPref, false);
|
| + EXPECT_FALSE(prefs->GetBoolean(kBoolPref));
|
| EXPECT_FALSE(boolean.GetValue());
|
| EXPECT_FALSE(*boolean);
|
|
|
| // Test int
|
| IntegerPrefMember integer;
|
| - integer.Init(kIntPref, &prefs, NULL);
|
| + integer.Init(kIntPref, prefs.get(), NULL);
|
|
|
| // Check the defaults
|
| - EXPECT_EQ(0, prefs.GetInteger(kIntPref));
|
| + EXPECT_EQ(0, prefs->GetInteger(kIntPref));
|
| EXPECT_EQ(0, integer.GetValue());
|
| EXPECT_EQ(0, *integer);
|
|
|
| // Try changing through the member variable.
|
| integer.SetValue(5);
|
| EXPECT_EQ(5, integer.GetValue());
|
| - EXPECT_EQ(5, prefs.GetInteger(kIntPref));
|
| + EXPECT_EQ(5, prefs->GetInteger(kIntPref));
|
| EXPECT_EQ(5, *integer);
|
|
|
| // Try changing back through the pref.
|
| - prefs.SetInteger(kIntPref, 2);
|
| - EXPECT_EQ(2, prefs.GetInteger(kIntPref));
|
| + prefs->SetInteger(kIntPref, 2);
|
| + EXPECT_EQ(2, prefs->GetInteger(kIntPref));
|
| EXPECT_EQ(2, integer.GetValue());
|
| EXPECT_EQ(2, *integer);
|
|
|
| // Test real (double)
|
| RealPrefMember real;
|
| - real.Init(kRealPref, &prefs, NULL);
|
| + real.Init(kRealPref, prefs.get(), NULL);
|
|
|
| // Check the defaults
|
| - EXPECT_EQ(0.0, prefs.GetReal(kRealPref));
|
| + EXPECT_EQ(0.0, prefs->GetReal(kRealPref));
|
| EXPECT_EQ(0.0, real.GetValue());
|
| EXPECT_EQ(0.0, *real);
|
|
|
| // Try changing through the member variable.
|
| real.SetValue(1.0);
|
| EXPECT_EQ(1.0, real.GetValue());
|
| - EXPECT_EQ(1.0, prefs.GetReal(kRealPref));
|
| + EXPECT_EQ(1.0, prefs->GetReal(kRealPref));
|
| EXPECT_EQ(1.0, *real);
|
|
|
| // Try changing back through the pref.
|
| - prefs.SetReal(kRealPref, 3.0);
|
| - EXPECT_EQ(3.0, prefs.GetReal(kRealPref));
|
| + prefs->SetReal(kRealPref, 3.0);
|
| + EXPECT_EQ(3.0, prefs->GetReal(kRealPref));
|
| EXPECT_EQ(3.0, real.GetValue());
|
| EXPECT_EQ(3.0, *real);
|
|
|
| // Test string
|
| StringPrefMember string;
|
| - string.Init(kStringPref, &prefs, NULL);
|
| + string.Init(kStringPref, prefs.get(), NULL);
|
|
|
| // Check the defaults
|
| - EXPECT_EQ("default", prefs.GetString(kStringPref));
|
| + EXPECT_EQ("default", prefs->GetString(kStringPref));
|
| EXPECT_EQ("default", string.GetValue());
|
| EXPECT_EQ("default", *string);
|
|
|
| // Try changing through the member variable.
|
| string.SetValue("foo");
|
| EXPECT_EQ("foo", string.GetValue());
|
| - EXPECT_EQ("foo", prefs.GetString(kStringPref));
|
| + EXPECT_EQ("foo", prefs->GetString(kStringPref));
|
| EXPECT_EQ("foo", *string);
|
|
|
| // Try changing back through the pref.
|
| - prefs.SetString(kStringPref, "bar");
|
| - EXPECT_EQ("bar", prefs.GetString(kStringPref));
|
| + prefs->SetString(kStringPref, "bar");
|
| + EXPECT_EQ("bar", prefs->GetString(kStringPref));
|
| EXPECT_EQ("bar", string.GetValue());
|
| EXPECT_EQ("bar", *string);
|
| }
|
|
|
| TEST(PrefMemberTest, TwoPrefs) {
|
| // Make sure two RealPrefMembers stay in sync.
|
| - TestingPrefService prefs;
|
| - RegisterTestPrefs(&prefs);
|
| + scoped_ptr<TestingPrefService> prefs(
|
| + TestingPrefService::CreateTestingPrefService());
|
| + RegisterTestPrefs(prefs.get());
|
|
|
| RealPrefMember pref1;
|
| - pref1.Init(kRealPref, &prefs, NULL);
|
| + pref1.Init(kRealPref, prefs.get(), NULL);
|
| RealPrefMember pref2;
|
| - pref2.Init(kRealPref, &prefs, NULL);
|
| + pref2.Init(kRealPref, prefs.get(), NULL);
|
|
|
| pref1.SetValue(2.3);
|
| EXPECT_EQ(2.3, *pref2);
|
| @@ -155,36 +157,37 @@ TEST(PrefMemberTest, TwoPrefs) {
|
| pref2.SetValue(3.5);
|
| EXPECT_EQ(3.5, *pref1);
|
|
|
| - prefs.SetReal(kRealPref, 4.2);
|
| + prefs->SetReal(kRealPref, 4.2);
|
| EXPECT_EQ(4.2, *pref1);
|
| EXPECT_EQ(4.2, *pref2);
|
| }
|
|
|
| TEST(PrefMemberTest, Observer) {
|
| - TestingPrefService prefs;
|
| - RegisterTestPrefs(&prefs);
|
| + scoped_ptr<TestingPrefService> prefs(
|
| + TestingPrefService::CreateTestingPrefService());
|
| + RegisterTestPrefs(prefs.get());
|
|
|
| - PrefMemberTestClass test_obj(&prefs);
|
| + PrefMemberTestClass test_obj(prefs.get());
|
| EXPECT_EQ("default", *test_obj.str_);
|
|
|
| // Calling SetValue should not fire the observer.
|
| test_obj.str_.SetValue("hello");
|
| EXPECT_EQ(0, test_obj.observe_cnt_);
|
| - EXPECT_EQ("hello", prefs.GetString(kStringPref));
|
| + EXPECT_EQ("hello", prefs->GetString(kStringPref));
|
|
|
| // Changing the pref does fire the observer.
|
| - prefs.SetString(kStringPref, "world");
|
| + prefs->SetString(kStringPref, "world");
|
| EXPECT_EQ(1, test_obj.observe_cnt_);
|
| EXPECT_EQ("world", *(test_obj.str_));
|
|
|
| // Not changing the value should not fire the observer.
|
| - prefs.SetString(kStringPref, "world");
|
| + prefs->SetString(kStringPref, "world");
|
| EXPECT_EQ(1, test_obj.observe_cnt_);
|
| EXPECT_EQ("world", *(test_obj.str_));
|
|
|
| - prefs.SetString(kStringPref, "hello");
|
| + prefs->SetString(kStringPref, "hello");
|
| EXPECT_EQ(2, test_obj.observe_cnt_);
|
| - EXPECT_EQ("hello", prefs.GetString(kStringPref));
|
| + EXPECT_EQ("hello", prefs->GetString(kStringPref));
|
| }
|
|
|
| TEST(PrefMemberTest, NoInit) {
|
|
|