This patch modifies some aspects of GCAPI in support of the new Chrome reactivation functions.

chrome/installer/gcapi/gcapi_reactivation_test.cc

 // Copyright (c) 2012 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 <string>
 
 #include "base/basictypes.h"
 #include "base/string_number_conversions.h"
 #include "base/stringprintf.h"
 #include "base/test/test_reg_util_win.h"
@@ skipping 107 matching lines @@
   }
 
  private:
   registry_util::RegistryOverrideManager override_manager_;
 };
 
 TEST_F(GCAPIReactivationTest, CheckSetReactivationBrandCode) {
   EXPECT_TRUE(SetReactivationBrandCode(L"GAGA", GCAPI_INVOKED_STANDARD_SHELL));
   EXPECT_EQ(L"GAGA", GetReactivationString(HKEY_CURRENT_USER));
 
   std::vector<std::wstring> check_codes;

[robertshield, 2012/03/23 17:52:16]

you can delete check_codes and just have a single 
EXPECT_TRUE(HasBeenReactivated());

[mgraboski, 2012/03/23 21:40:35]

Done.

   check_codes.push_back(L"GAGA");
-  EXPECT_TRUE(HasBeenReactivatedByBrandCodes(check_codes));
+  EXPECT_TRUE(HasBeenReactivated());
 
   check_codes.push_back(L"GOOGOO");
-  EXPECT_TRUE(HasBeenReactivatedByBrandCodes(check_codes));
+  EXPECT_TRUE(HasBeenReactivated());
 
   check_codes.erase(check_codes.begin());
-  EXPECT_FALSE(HasBeenReactivatedByBrandCodes(check_codes));
+  EXPECT_TRUE(HasBeenReactivated());
 }
 
 TEST_F(GCAPIReactivationTest, CanOfferReactivation_Basic) {
   const wchar_t* previous_brands[] = {L"GOOGOO", L"MAMA", L"DADA"};
   DWORD error;
 
   // We're not installed yet. Make sure CanOfferReactivation fails.
   EXPECT_FALSE(CanOfferReactivation(L"GAGA", arraysize(previous_brands),
                                     previous_brands,
                                     GCAPI_INVOKED_STANDARD_SHELL,
@@ skipping 69 matching lines @@
                                previous_brands, GCAPI_INVOKED_STANDARD_SHELL,
                                &error));
   EXPECT_EQ(L"GAGA", GetReactivationString(HKEY_CURRENT_USER));
 
   // Make sure we can't reactivate again:
   EXPECT_FALSE(ReactivateChrome(L"GAGA", arraysize(previous_brands),
                                 previous_brands, GCAPI_INVOKED_STANDARD_SHELL,
                                 &error));
   EXPECT_EQ(REACTIVATE_ERROR_ALREADY_REACTIVATED, error);
 
-  // Should still be able to reactivate under other brands:
-  EXPECT_TRUE(ReactivateChrome(L"MAMA", arraysize(previous_brands),
+  // Should not be able to reactivate under other brands:
+  EXPECT_FALSE(ReactivateChrome(L"MAMA", arraysize(previous_brands),
                                previous_brands, GCAPI_INVOKED_STANDARD_SHELL,
                                &error));
-  EXPECT_EQ(L"MAMA", GetReactivationString(HKEY_CURRENT_USER));
+  EXPECT_EQ(L"GAGA", GetReactivationString(HKEY_CURRENT_USER));
 
   // Validate that previous_brands are rejected:
   EXPECT_FALSE(ReactivateChrome(L"PFFT", arraysize(previous_brands),
                                 previous_brands, GCAPI_INVOKED_STANDARD_SHELL,
                                 &error));
   EXPECT_EQ(REACTIVATE_ERROR_ALREADY_REACTIVATED, error);
-  EXPECT_EQ(L"MAMA", GetReactivationString(HKEY_CURRENT_USER));
+  EXPECT_EQ(L"GAGA", GetReactivationString(HKEY_CURRENT_USER));
 }
 
 TEST_F(GCAPIReactivationTest, ExperimentLabelCheck) {
   const wchar_t* previous_brands[] = {L"GOOGOO", L"MAMA", L"DADA"};
   DWORD error;
 
   // Set us up as a candidate for reactivation.
   EXPECT_TRUE(SetChromeInstallMarker(HKEY_CURRENT_USER));
 
   Time hkcu_last_run = Time::NowFromSystemTime() -
       TimeDelta::FromDays(kReactivationMinDaysDormant);
   EXPECT_TRUE(SetLastRunTime(HKEY_CURRENT_USER,
                              hkcu_last_run.ToInternalValue()));
 
   EXPECT_TRUE(ReactivateChrome(L"GAGA", arraysize(previous_brands),
                                previous_brands, GCAPI_INVOKED_STANDARD_SHELL,
                                &error));
   EXPECT_EQ(L"GAGA", GetReactivationString(HKEY_CURRENT_USER));
 
   EXPECT_TRUE(HasExperimentLabels(HKEY_CURRENT_USER));
 }