Clear client id prefs when opting out of UMA.

chrome/installer/util/google_update_settings_unittest.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 "chrome/installer/util/google_update_settings.h"
 
 #include <windows.h>
 #include <shlwapi.h>  // For SHDeleteKey.
 #include <stddef.h>
 
@@ skipping 1253 matching lines @@
 void CollectStatsConsent::TearDownTestCase() {
   delete chrome_version_key_;
   delete chrome_state_key_;
   delete chrome_state_medium_key_;
   delete binaries_state_key_;
   delete binaries_state_medium_key_;
 }
 
 // Install the registry override and apply the settings to the registry.
 void CollectStatsConsent::SetUp() {
+  // Override both HKLM and HKCU as tests may touch either/both.
+  override_manager_.OverrideRegistry(HKEY_LOCAL_MACHINE);
+  override_manager_.OverrideRegistry(HKEY_CURRENT_USER);
+
   const StatsState& stats_state = GetParam();
   const HKEY root_key = stats_state.root_key();
-  base::string16 reg_temp_name(
-      stats_state.system_level() ? L"HKLM_" : L"HKCU_");
-  reg_temp_name += L"CollectStatsConsent";
-  override_manager_.OverrideRegistry(root_key);
-
   if (stats_state.multi_install()) {
     MakeChromeMultiInstall(root_key);
     ApplySetting(stats_state.state_value(), root_key, *binaries_state_key_);
     ApplySetting(stats_state.state_medium_value(), root_key,
                  *binaries_state_medium_key_);
   } else {
     ApplySetting(stats_state.state_value(), root_key, *chrome_state_key_);
     ApplySetting(stats_state.state_medium_value(), root_key,
                  *chrome_state_medium_key_);
   }
@@ skipping 35 matching lines @@
                     GetParam().system_level()));
   } else {
     EXPECT_FALSE(GoogleUpdateSettings::GetCollectStatsConsentAtLevel(
                      GetParam().system_level()));
   }
 }
 
 // Test that stats consent can be flipped to the opposite setting, that the new
 // setting takes affect, and that the correct registry location is modified.
 TEST_P(CollectStatsConsent, SetCollectStatsConsentAtLevel) {
+  // When testing revoking consent, verify that backup client info is cleared.
+  // To do so, first add some backup client info.
+  if (GetParam().is_consent_granted()) {
+    metrics::ClientInfo client_info;
+    client_info.client_id = "01234567-89ab-cdef-fedc-ba9876543210";
+    client_info.installation_date = 123;
+    client_info.reporting_enabled_date = 345;
+    GoogleUpdateSettings::StoreMetricsClientInfo(client_info);
+  }
+
   EXPECT_TRUE(GoogleUpdateSettings::SetCollectStatsConsentAtLevel(
                   GetParam().system_level(),
                   !GetParam().is_consent_granted()));
+
   const base::string16* const reg_keys[] = {
     chrome_state_key_,
     chrome_state_medium_key_,
     binaries_state_key_,
     binaries_state_medium_key_,
   };
   int key_index = ((GetParam().system_level() ? 1 : 0) +
                    (GetParam().multi_install() ? 2 : 0));
   const base::string16& reg_key = *reg_keys[key_index];
   DWORD value = 0;
   EXPECT_EQ(
       ERROR_SUCCESS,
       RegKey(GetParam().root_key(), reg_key.c_str(),
              KEY_QUERY_VALUE).ReadValueDW(google_update::kRegUsageStatsField,
                                           &value));
   if (GetParam().is_consent_granted()) {
     EXPECT_FALSE(GoogleUpdateSettings::GetCollectStatsConsentAtLevel(
                      GetParam().system_level()));
     EXPECT_EQ(0UL, value);
   } else {
     EXPECT_TRUE(GoogleUpdateSettings::GetCollectStatsConsentAtLevel(
                     GetParam().system_level()));
     EXPECT_EQ(1UL, value);
+    // Verify that backup client info has been cleared.
+    EXPECT_FALSE(GoogleUpdateSettings::LoadMetricsClientInfo());
   }
 }
 
 INSTANTIATE_TEST_CASE_P(
     UserLevelSingleInstall,
     CollectStatsConsent,
     ::testing::Values(
         StatsState(StatsState::kUserLevel, StatsState::SINGLE_INSTALL,
                    StatsState::NO_SETTING),
         StatsState(StatsState::kUserLevel, StatsState::SINGLE_INSTALL,
@@ skipping 47 matching lines @@
         StatsState(StatsState::kSystemLevel, StatsState::MULTI_INSTALL,
                    StatsState::FALSE_SETTING, StatsState::FALSE_SETTING),
         StatsState(StatsState::kSystemLevel, StatsState::MULTI_INSTALL,
                    StatsState::FALSE_SETTING, StatsState::TRUE_SETTING),
         StatsState(StatsState::kSystemLevel, StatsState::MULTI_INSTALL,
                    StatsState::TRUE_SETTING, StatsState::NO_SETTING),
         StatsState(StatsState::kSystemLevel, StatsState::MULTI_INSTALL,
                    StatsState::TRUE_SETTING, StatsState::FALSE_SETTING),
         StatsState(StatsState::kSystemLevel, StatsState::MULTI_INSTALL,
                    StatsState::TRUE_SETTING, StatsState::TRUE_SETTING)));