Resetting metrics ids on clump detection.

chrome/browser/metrics/metrics_service_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/browser/metrics/metrics_service.h"
 
 #include <ctype.h>
 #include <string>
 
 #include "base/command_line.h"
@@ skipping 356 matching lines @@
   GetLocalState()->SetBoolean(crash_pref, true);
   EXPECT_TRUE(MetricsServiceHelper::IsCrashReportingEnabled());
   GetLocalState()->ClearPref(crash_pref);
   EXPECT_FALSE(MetricsServiceHelper::IsCrashReportingEnabled());
 #endif // !defined(OS_CHROMEOS)
 #else // defined(GOOGLE_CHROME_BUILD)
   // Chromium branded browsers never have crash reporting enabled.
   EXPECT_FALSE(MetricsServiceHelper::IsCrashReportingEnabled());
 #endif // defined(GOOGLE_CHROME_BUILD)
 }
+
+TEST_F(MetricsServiceTest, ResetMetricsIDs) {
+  // Ensuere that the low entropy source and client id have been generated.

[Alexei Svitkine (slow), 2014/03/28 20:05:47]

Nit: Ensure

[jwd, 2014/03/28 20:18:13]

Done.

+  {
+    MetricsService service;
+    service.ForceClientIdCreation();
+    service.GetLowEntropySource();
+  }
+
+  std::string old_client_id =
+      GetLocalState()->GetString(prefs::kMetricsClientID);
+  int old_low_entropy_value =
+      GetLocalState()->GetInteger(prefs::kMetricsLowEntropySource);
+
+  // Set the reset pref to cause the IDs to be reset.
+  GetLocalState()->SetBoolean(prefs::kMetricsResetIds, true);
+
+  // Cause the actual reset to happen.
+  {
+    MetricsService service;
+    service.ForceClientIdCreation();
+    EXPECT_TRUE(service.metrics_ids_reset_check_performed_);
+    EXPECT_NE(old_client_id, service.client_id_);
+
+    service.GetLowEntropySource();
+    EXPECT_NE(old_low_entropy_value, service.low_entropy_source_);

[Alexei Svitkine (slow), 2014/03/28 20:05:47]

This will be flaky - since it could happen that we get the same low entropy
source value. Let's just do the check for the client id.

[jwd, 2014/03/28 20:18:13]

Done.

+
+    EXPECT_FALSE(GetLocalState()->GetBoolean(prefs::kMetricsResetIds));
+  }
+
+  std::string new_client_id =
+      GetLocalState()->GetString(prefs::kMetricsClientID);
+  int new_low_entropy_value =
+      GetLocalState()->GetInteger(prefs::kMetricsLowEntropySource);
+
+  EXPECT_NE(old_client_id, new_client_id);
+  EXPECT_NE(old_low_entropy_value, new_low_entropy_value);
+}