Make GetDistroBooleanPreference return troolean value.

BUG=23649
TEST=installer_util_unittests, setup_unittests

Review URL: http://codereview.chromium.org/269107

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29423 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 50 insertions, 27 deletions

diff --git a/chrome/installer/util/master_preferences_unittest.cc b/chrome/installer/util/master_preferences_unittest.cc
index 18f3a6b..ad827eb 100644
--- a/chrome/installer/util/master_preferences_unittest.cc
+++ b/chrome/installer/util/master_preferences_unittest.cc
@@ -56,40 +56,56 @@ TEST(MasterPreferencesTest, ParseDistroParams) {
   scoped_ptr<DictionaryValue> prefs(
       installer_util::ParseDistributionPreferences(prefs_file));
   EXPECT_TRUE(prefs.get() != NULL);
+  bool value = true;
   EXPECT_TRUE(installer_util::GetDistroBooleanPreference(prefs.get(),
-      installer_util::master_preferences::kDistroSkipFirstRunPref));
+      installer_util::master_preferences::kDistroSkipFirstRunPref, &value) &&
+      value);
   EXPECT_TRUE(installer_util::GetDistroBooleanPreference(prefs.get(),
-      installer_util::master_preferences::kDistroShowWelcomePage));
+      installer_util::master_preferences::kDistroShowWelcomePage, &value) &&
+      value);
 
   EXPECT_TRUE(installer_util::GetDistroBooleanPreference(prefs.get(),
-      installer_util::master_preferences::kDistroImportSearchPref));
+      installer_util::master_preferences::kDistroImportSearchPref, &value) &&
+      value);
   EXPECT_TRUE(installer_util::GetDistroBooleanPreference(prefs.get(),
-      installer_util::master_preferences::kDistroImportHistoryPref));
+      installer_util::master_preferences::kDistroImportHistoryPref, &value) &&
+      value);
   EXPECT_TRUE(installer_util::GetDistroBooleanPreference(prefs.get(),
-      installer_util::master_preferences::kDistroImportBookmarksPref));
+      installer_util::master_preferences::kDistroImportBookmarksPref, &value) &&
+      value);
   EXPECT_TRUE(installer_util::GetDistroBooleanPreference(prefs.get(),
-      installer_util::master_preferences::kDistroImportHomePagePref));
+      installer_util::master_preferences::kDistroImportHomePagePref, &value) &&
+      value);
 
   EXPECT_TRUE(installer_util::GetDistroBooleanPreference(prefs.get(),
-      installer_util::master_preferences::kCreateAllShortcuts));
+      installer_util::master_preferences::kCreateAllShortcuts, &value) &&
+      value);
   EXPECT_TRUE(installer_util::GetDistroBooleanPreference(prefs.get(),
-      installer_util::master_preferences::kDoNotLaunchChrome));
+      installer_util::master_preferences::kDoNotLaunchChrome, &value) &&
+      value);
   EXPECT_TRUE(installer_util::GetDistroBooleanPreference(prefs.get(),
-      installer_util::master_preferences::kMakeChromeDefault));
+      installer_util::master_preferences::kMakeChromeDefault, &value) &&
+      value);
   EXPECT_TRUE(installer_util::GetDistroBooleanPreference(prefs.get(),
-      installer_util::master_preferences::kMakeChromeDefaultForUser));
+      installer_util::master_preferences::kMakeChromeDefaultForUser, &value) &&
+      value);
 
   EXPECT_TRUE(installer_util::GetDistroBooleanPreference(prefs.get(),
-      installer_util::master_preferences::kSystemLevel));
+      installer_util::master_preferences::kSystemLevel, &value) &&
+      value);
   EXPECT_TRUE(installer_util::GetDistroBooleanPreference(prefs.get(),
-      installer_util::master_preferences::kVerboseLogging));
+      installer_util::master_preferences::kVerboseLogging, &value) &&
+      value);
   EXPECT_TRUE(installer_util::GetDistroBooleanPreference(prefs.get(),
-      installer_util::master_preferences::kRequireEula));
+      installer_util::master_preferences::kRequireEula, &value) &&
+      value);
   EXPECT_TRUE(installer_util::GetDistroBooleanPreference(prefs.get(),
-      installer_util::master_preferences::kAltShortcutText));
+      installer_util::master_preferences::kAltShortcutText, &value) &&
+      value);
 
   EXPECT_TRUE(installer_util::GetDistroBooleanPreference(prefs.get(),
-      installer_util::master_preferences::kAltFirstRunBubble));
+      installer_util::master_preferences::kAltFirstRunBubble, &value) &&
+      value);
   int icon_index = 0;
   EXPECT_TRUE(installer_util::GetDistroIntegerPreference(prefs.get(),
       installer_util::master_preferences::kChromeShortcutIconIndex,
@@ -121,30 +137,37 @@ TEST(MasterPreferencesTest, ParseMissingDistroParams) {
   scoped_ptr<DictionaryValue> prefs(
       installer_util::ParseDistributionPreferences(prefs_file));
   EXPECT_TRUE(prefs.get() != NULL);
+  bool value = false;
   EXPECT_TRUE(installer_util::GetDistroBooleanPreference(prefs.get(),
-      installer_util::master_preferences::kDistroSkipFirstRunPref));
+      installer_util::master_preferences::kDistroSkipFirstRunPref, &value) &&
+      value);
   EXPECT_FALSE(installer_util::GetDistroBooleanPreference(prefs.get(),
-      installer_util::master_preferences::kDistroShowWelcomePage));
+      installer_util::master_preferences::kDistroShowWelcomePage, &value));
 
   EXPECT_TRUE(installer_util::GetDistroBooleanPreference(prefs.get(),
-      installer_util::master_preferences::kDistroImportSearchPref));
+      installer_util::master_preferences::kDistroImportSearchPref, &value) &&
+      value);
   EXPECT_FALSE(installer_util::GetDistroBooleanPreference(prefs.get(),
-      installer_util::master_preferences::kDistroImportHistoryPref));
-  EXPECT_FALSE(installer_util::GetDistroBooleanPreference(prefs.get(),
-      installer_util::master_preferences::kDistroImportBookmarksPref));
+      installer_util::master_preferences::kDistroImportHistoryPref, &value));
+  EXPECT_TRUE(installer_util::GetDistroBooleanPreference(prefs.get(),
+      installer_util::master_preferences::kDistroImportBookmarksPref, &value));
+  EXPECT_FALSE(value);
   EXPECT_FALSE(installer_util::GetDistroBooleanPreference(prefs.get(),
-      installer_util::master_preferences::kDistroImportHomePagePref));
+      installer_util::master_preferences::kDistroImportHomePagePref, &value));
 
   EXPECT_TRUE(installer_util::GetDistroBooleanPreference(prefs.get(),
-      installer_util::master_preferences::kCreateAllShortcuts));
+      installer_util::master_preferences::kCreateAllShortcuts, &value) &&
+      value);
   EXPECT_TRUE(installer_util::GetDistroBooleanPreference(prefs.get(),
-      installer_util::master_preferences::kDoNotLaunchChrome));
+      installer_util::master_preferences::kDoNotLaunchChrome, &value) &&
+      value);
   EXPECT_FALSE(installer_util::GetDistroBooleanPreference(prefs.get(),
-      installer_util::master_preferences::kDoNotRegisterForUpdateLaunch));
+      installer_util::master_preferences::kDoNotRegisterForUpdateLaunch,
+      &value));
   EXPECT_FALSE(installer_util::GetDistroBooleanPreference(prefs.get(),
-      installer_util::master_preferences::kMakeChromeDefault));
+      installer_util::master_preferences::kMakeChromeDefault, &value));
   EXPECT_FALSE(installer_util::GetDistroBooleanPreference(prefs.get(),
-      installer_util::master_preferences::kMakeChromeDefaultForUser));
+      installer_util::master_preferences::kMakeChromeDefaultForUser, &value));
 
   int icon_index = 0;
   EXPECT_FALSE(installer_util::GetDistroIntegerPreference(prefs.get(),