One-time reset of UMA client ids and low entropy source.

chrome/common/pref_names.cc

Index: chrome/common/pref_names.cc
===================================================================
--- chrome/common/pref_names.cc	(revision 259917)
+++ chrome/common/pref_names.cc	(working copy)
@@ -1362,17 +1362,24 @@
 const char kEulaAccepted[] = "EulaAccepted";
 
 // The metrics client GUID, entropy source and session ID.
-const char kMetricsClientID[] = "user_experience_metrics.client_id";
+const char kMetricsClientID[] = "user_experience_metrics.client_id2";

[rkaplow, 2014/03/28 17:20:26]

Understand this is temp but would still add a comment explaining the 2s here.

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

Agree about the comment, though the 2 is here to stay.

Added comment to explain this.

+// Date/time when the user opted in to UMA (local machine time, stored as a
+// 64-bit time_t value).
+const char kMetricsClientIDTimestamp[] =
+    "user_experience_metrics.client_id_timestamp";
 const char kMetricsSessionID[] = "user_experience_metrics.session_id";
 const char kMetricsLowEntropySource[] =
-    "user_experience_metrics.low_entropy_source";
+    "user_experience_metrics.low_entropy_source2";
 const char kMetricsPermutedEntropyCache[] =
     "user_experience_metrics.permuted_entropy_cache";
 
-// Date/time when the current metrics profile ID was created
-// (which hopefully corresponds to first run).
-const char kMetricsClientIDTimestamp[] =
-    "user_experience_metrics.client_id_timestamp";
+// Old client id and low entropy source values, cleared the first time this
+// version is launched.
+// TODO(asvitkine): Delete these after a few releases have gone by and old
+// values have been cleaned up.
+const char kMetricsOldClientID[] = "user_experience_metrics.client_id";
+const char kMetricsOldLowEntropySource[] =
+    "user_experience_metrics.low_entropy_source";
 
 // Boolean that specifies whether or not crash reporting and metrics reporting
 // are sent over the network for analysis.