Switching the no-finch default behavior to enable the blacklist

chrome/browser/chrome_elf_init_unittest_win.cc

Index: chrome/browser/chrome_elf_init_unittest_win.cc
diff --git a/chrome/browser/chrome_elf_init_unittest_win.cc b/chrome/browser/chrome_elf_init_unittest_win.cc
index b7cee8af896a99a6487b227ea91689dc24c476e9..5fcb16638994f67a51801df65ddbf401ebd41d38 100644
--- a/chrome/browser/chrome_elf_init_unittest_win.cc
+++ b/chrome/browser/chrome_elf_init_unittest_win.cc
@@ -54,22 +54,23 @@ class ChromeBlacklistTrialTest : public testing::Test {
   DISALLOW_COPY_AND_ASSIGN(ChromeBlacklistTrialTest);
 };
 
-
-// Ensure that the default trial deletes any existing blacklist beacons.
+// Ensure that the default trial sets up the blacklist beacons.
 TEST_F(ChromeBlacklistTrialTest, DefaultRun) {

[Cait (Slow), 2014/05/15 18:43:19]

This looks a lot like the test below it "VerifyFirstRun" now  -- is it worth
adding a test to confirm that users in the "BlacklistDisabled" Finch group are
still having their blacklists disabled (if there are any users in that group any
more)?

[csharp, 2014/05/15 20:09:39]

Yup, that is probably a good idea :)

[krstnmnlsn, 2014/05/16 14:15:03]

Done.

   // Set some dummy values as beacons.
   blacklist_registry_key_->WriteValue(blacklist::kBeaconState,
-                                      blacklist::BLACKLIST_ENABLED);
+                                      blacklist::BLACKLIST_DISABLED);
   blacklist_registry_key_->WriteValue(blacklist::kBeaconVersion, L"Data");
 
-  // This setup code should result in the default group, which should remove
-  // all the beacon values.
+  // This setup code should result in the default group, which should have
+  // the blacklist set up.
   InitializeChromeElf();
 
-  // Ensure that invalid values are returned to indicate that the
-  // beacon values are gone.
-  ASSERT_EQ(blacklist::BLACKLIST_STATE_MAX, GetBlacklistState());
-  ASSERT_EQ(base::string16(), GetBlacklistVersion());
+  // Ensure the beacon values are now correct, indicating the
+  // blacklist beacon was setup.
+  ASSERT_EQ(blacklist::BLACKLIST_ENABLED, GetBlacklistState());
+  chrome::VersionInfo version_info;
+  base::string16 version(base::UTF8ToUTF16(version_info.Version()));
+  ASSERT_EQ(version, GetBlacklistVersion());
 }
 
 TEST_F(ChromeBlacklistTrialTest, VerifyFirstRun) {