Third patch in getting rid of caching MessageLoop pointers and always using C...

chrome/common/pref_member_unittest.cc

 // Copyright (c) 2006-2008 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "base/file_path.h"
 #include "chrome/common/notification_service.h"
 #include "chrome/common/pref_member.h"
 #include "chrome/common/pref_service.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
@@ skipping 32 matching lines @@
   StringPrefMember str_;
   int observe_cnt_;
 
  private:
   PrefService* prefs_;
 };
 
 }  // anonymous namespace
 
 TEST(PrefMemberTest, BasicGetAndSet) {
-  PrefService prefs(FilePath(), NULL);
+  PrefService prefs((FilePath()));
   RegisterTestPrefs(&prefs);
 
   // Test bool
   BooleanPrefMember boolean;
   boolean.Init(kBoolPref, &prefs, NULL);
 
   // Check the defaults
   EXPECT_FALSE(prefs.GetBoolean(kBoolPref));
   EXPECT_FALSE(boolean.GetValue());
   EXPECT_FALSE(*boolean);
@@ skipping 69 matching lines @@
 
   // Try changing back through the pref.
   prefs.SetString(kStringPref, L"bar");
   EXPECT_EQ(L"bar", prefs.GetString(kStringPref));
   EXPECT_EQ(L"bar", string.GetValue());
   EXPECT_EQ(L"bar", *string);
 }
 
 TEST(PrefMemberTest, TwoPrefs) {
   // Make sure two RealPrefMembers stay in sync.
-  PrefService prefs(FilePath(), NULL);
+  PrefService prefs((FilePath()));
   RegisterTestPrefs(&prefs);
 
   RealPrefMember pref1;
   pref1.Init(kRealPref, &prefs, NULL);
   RealPrefMember pref2;
   pref2.Init(kRealPref, &prefs, NULL);
 
   pref1.SetValue(2.3);
   EXPECT_EQ(2.3, *pref2);
 
   pref2.SetValue(3.5);
   EXPECT_EQ(3.5, *pref1);
 
   prefs.SetReal(kRealPref, 4.2);
   EXPECT_EQ(4.2, *pref1);
   EXPECT_EQ(4.2, *pref2);
 }
 
 TEST(PrefMemberTest, Observer) {
-  PrefService prefs(FilePath(), NULL);
+  PrefService prefs((FilePath()));
   RegisterTestPrefs(&prefs);
 
   PrefMemberTestClass test_obj(&prefs);
   EXPECT_EQ(L"default", *test_obj.str_);
 
   // Calling SetValue should not fire the observer.
   test_obj.str_.SetValue(L"hello");
   EXPECT_EQ(0, test_obj.observe_cnt_);
   EXPECT_EQ(L"hello", prefs.GetString(kStringPref));
 
   // Changing the pref does fire the observer.
   prefs.SetString(kStringPref, L"world");
   EXPECT_EQ(1, test_obj.observe_cnt_);
   EXPECT_EQ(L"world", *(test_obj.str_));
 
   // Not changing the value should not fire the observer.
   prefs.SetString(kStringPref, L"world");
   EXPECT_EQ(1, test_obj.observe_cnt_);
   EXPECT_EQ(L"world", *(test_obj.str_));
 
   prefs.SetString(kStringPref, L"hello");
   EXPECT_EQ(2, test_obj.observe_cnt_);
   EXPECT_EQ(L"hello", prefs.GetString(kStringPref));
 }
 
 TEST(PrefMemberTest, NoInit) {
   // Make sure not calling Init on a PrefMember doesn't cause problems.
   IntegerPrefMember pref;
 }