Resetting metrics ids on clump detection.

chrome/browser/metrics/metrics_service.h

Index: chrome/browser/metrics/metrics_service.h
diff --git a/chrome/browser/metrics/metrics_service.h b/chrome/browser/metrics/metrics_service.h
index f50934d0db5d53e72d624dcde35a735a02e18a23..62aacd5dc4912ab87764883b2ce1472cc55ff995 100644
--- a/chrome/browser/metrics/metrics_service.h
+++ b/chrome/browser/metrics/metrics_service.h
@@ -142,6 +142,9 @@ class MetricsService
   // is enabled.
   void InitializeMetricsRecordingState(ReportingState reporting_state);
 
+  // Initializes low entropy source, and, if the client has
+  void InitializeVariationsState(bool recording_enabled);

[Alexei Svitkine (slow), 2014/03/28 15:00:17]

It's not really variations state, since UMA client id isn't just for variations.

Maybe InitializeMetricsIDs()?

[jwd, 2014/03/28 16:23:50]

Done.

+
   // Starts the metrics system, turning on recording and uploading of metrics.
   // Should be called when starting up with metrics enabled, or when metrics
   // are turned on.
@@ -293,6 +296,8 @@ class MetricsService
   // should not be called more than once.
   void CheckForClonedInstall();
 
+  void ResetVariationsState();

[Alexei Svitkine (slow), 2014/03/28 15:00:17]

Private?

[jwd, 2014/03/28 16:23:50]

Done.

+
  private:
   // The MetricsService has a lifecycle that is stored as a state.
   // See metrics_service.cc for description of this lifecycle.
@@ -523,6 +528,8 @@ class MetricsService
 
   content::NotificationRegistrar registrar_;
 
+  bool variations_state_initialized_;
+
   // Indicate whether recording and reporting are currently happening.
   // These should not be set directly, but by calling SetRecording and
   // SetReporting.