Use a Finch Experiment to control the Browser Blacklist

chrome_elf/blacklist/test/blacklist_test.cc

Index: chrome_elf/blacklist/test/blacklist_test.cc
diff --git a/chrome_elf/blacklist/test/blacklist_test.cc b/chrome_elf/blacklist/test/blacklist_test.cc
index dbb662f97114684b3bd2d2aa45d6156b41082bd5..66027f6e38ec411d4a8d1b1ede034cd7237e87d8 100644
--- a/chrome_elf/blacklist/test/blacklist_test.cc
+++ b/chrome_elf/blacklist/test/blacklist_test.cc
@@ -38,8 +38,8 @@ class BlacklistTest : public testing::Test {
     // Force an import from blacklist_test_main_dll.
     InitBlacklistTestDll();
 
-    // Ensure that the beacon state starts off cleared.
-    blacklist::ClearBeacon();
+    // Ensure that the beacon state starts off enabled.
+    blacklist::ResetBeacon();
   }
 
   virtual void TearDown() {
@@ -52,17 +52,14 @@ TEST_F(BlacklistTest, Beacon) {
   registry_util::RegistryOverrideManager override_manager;
   override_manager.OverrideRegistry(HKEY_CURRENT_USER, L"beacon_test");
 
-  // First call should succeed as the beacon is newly created.
-  EXPECT_TRUE(blacklist::CreateBeacon());
+  // First call should find the beacon and reset it.
+  EXPECT_TRUE(blacklist::ResetBeacon());
 
-  // Second call should fail indicating the beacon already existed.
-  EXPECT_FALSE(blacklist::CreateBeacon());
+  // First call should succeed as the beacon is enabled.
+  EXPECT_TRUE(blacklist::LeaveSetupBeacon());
 
-  // First call should find the beacon and delete it.
-  EXPECT_TRUE(blacklist::ClearBeacon());
-
-  // Second call should fail to find the beacon and delete it.
-  EXPECT_FALSE(blacklist::ClearBeacon());
+  // Second call should fail indicating the beacon wasn't set as enabled.
+  EXPECT_FALSE(blacklist::LeaveSetupBeacon());

[robertshield, 2014/01/02 20:42:22]

Also check that ResetBeacon works after calling LeaveSetupBeacon.

[csharp, 2014/01/02 21:57:59]

Done.

 }
 
 TEST_F(BlacklistTest, AddAndRemoveModules) {