[UMA] Rename the misleadinglingly named "install_date" field in the UMA uploads.

Review URL: https://chromiumcodereview.appspot.com/11344022

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@164792 0039d316-1c4b-4281-b951-d872f2087c98

2 files changed, 10 insertions, 8 deletions

diff --git a/chrome/browser/metrics/metrics_log.cc b/chrome/browser/metrics/metrics_log.cc
index ce22188..7ff5b5a 100644
--- a/chrome/browser/metrics/metrics_log.cc
+++ b/chrome/browser/metrics/metrics_log.cc
@@ -70,7 +70,7 @@ namespace {
 
 // Returns the date at which the current metrics client ID was created as
 // a string containing milliseconds since the epoch, or "0" if none was found.
-std::string GetInstallDate(PrefService* pref) {
+std::string GetMetricsEnabledDate(PrefService* pref) {
   if (!pref) {
     NOTREACHED();
     return "0";
@@ -644,7 +644,7 @@ void MetricsLog::WriteInstallElement() {
   // Write the XML version.
   // We'll write the protobuf version in RecordEnvironmentProto().
   OPEN_ELEMENT_FOR_SCOPE("install");
-  WriteAttribute("installdate", GetInstallDate(GetPrefService()));
+  WriteAttribute("installdate", GetMetricsEnabledDate(GetPrefService()));
   WriteIntAttribute("buildid", 0);  // We're using appversion instead.
 }
 
@@ -757,11 +757,11 @@ void MetricsLog::RecordEnvironmentProto(
     const std::vector<webkit::WebPluginInfo>& plugin_list,
     const GoogleUpdateMetrics& google_update_metrics) {
   SystemProfileProto* system_profile = uma_proto()->mutable_system_profile();
-  int install_date;
-  bool success = base::StringToInt(GetInstallDate(GetPrefService()),
-                                   &install_date);
+  int enabled_date;
+  bool success = base::StringToInt(GetMetricsEnabledDate(GetPrefService()),
+                                   &enabled_date);
   DCHECK(success);
-  system_profile->set_install_date(install_date);
+  system_profile->set_uma_enabled_date(enabled_date);
 
   system_profile->set_application_locale(
       content::GetContentClient()->browser()->GetApplicationLocale());
diff --git a/chrome/common/metrics/proto/system_profile.proto b/chrome/common/metrics/proto/system_profile.proto
index 420ec02..bf25865 100644
--- a/chrome/common/metrics/proto/system_profile.proto
+++ b/chrome/common/metrics/proto/system_profile.proto
@@ -50,8 +50,10 @@ message SystemProfileProto {
   }
   optional Channel channel = 10;
 
-  // The date the user installed the browser, in seconds since the epoch.
-  optional int64 install_date = 3;
+  // The date the user enabled UMA, in seconds since the epoch.
+  // If the user has toggled the UMA enabled state multiple times, this will
+  // be the most recent date on which UMA was enabled.
+  optional int64 uma_enabled_date = 3;
 
   // The user's selected application locale, i.e. the user interface language.
   // The locale includes a language code and, possibly, also a country code,