Review URL: http://codereview.chromium.org/62160

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

2 files changed, 5 insertions, 20 deletions

diff --git a/chrome/installer/util/google_chrome_distribution.cc b/chrome/installer/util/google_chrome_distribution.cc
index 76ea297..920ea47 100644
--- a/chrome/installer/util/google_chrome_distribution.cc
+++ b/chrome/installer/util/google_chrome_distribution.cc
@@ -101,21 +101,6 @@ bool GoogleChromeDistribution::ExtractUninstallMetrics(
     return false;
   }
 
-  // We use the creation date of the metric's client id as the installation
-  // date, as this is fairly close to the first-run date for those who
-  // opt-in to metrics from the get-go. Slightly more accurate would be to
-  // have setup.exe write something somewhere at installation time.
-  // As is, the absence of this field implies that the browser was never
-  // run with an opt-in to metrics selection.
-  std::wstring installation_date;
-  if (root.GetString(prefs::kMetricsClientIDTimestamp, &installation_date)) {
-    uninstall_metrics_string->append(L"&");
-    uninstall_metrics_string->append(
-        installer_util::kUninstallInstallationDate);
-    uninstall_metrics_string->append(L"=");
-    uninstall_metrics_string->append(installation_date);
-  }
-
   return true;
 }
 
diff --git a/chrome/installer/util/google_chrome_distribution_unittest.cc b/chrome/installer/util/google_chrome_distribution_unittest.cc
index bd13cdc..cca9e52 100644
--- a/chrome/installer/util/google_chrome_distribution_unittest.cc
+++ b/chrome/installer/util/google_chrome_distribution_unittest.cc
@@ -194,7 +194,8 @@ TEST_F(GoogleChromeDistributionTest, TestExtractUninstallMetrics) {
       "    \"last_observed_running_time_sec\": \"1235341183\","
       "    \"launch_count\": \"11\","
       "    \"page_load_count\": \"68\","
-      "    \"uptime_sec\": \"809\"\n"
+      "    \"uptime_sec\": \"809\","
+      "    \"installation_date2\": \"1235341141\"\n"
       "  },\n"
       "  \"blah\": {\n"
       "    \"this_sentence_is_true\": false\n"
@@ -207,11 +208,11 @@ TEST_F(GoogleChromeDistributionTest, TestExtractUninstallMetrics) {
 
   // The URL string we expect to be generated from said make-believe file.
   std::wstring expected_url_string(
+      L"&installation_date2=1235341141"
       L"&last_launch_time_sec=1235341118"
       L"&last_observed_running_time_sec=1235341183"
-      L"&launch_count=11&page_load_count=68&uptime_sec=809&");
-  expected_url_string += installer_util::kUninstallInstallationDate;
-  expected_url_string += L"=1234567890";
+      L"&launch_count=11&page_load_count=68"
+      L"&uptime_sec=809");
 
   JSONStringValueSerializer json_deserializer(pref_string);
   std::string error_message;
@@ -294,4 +295,3 @@ TEST(MasterPreferences, ParseDistroParams) {
   EXPECT_TRUE(result & installer_util::MASTER_PROFILE_ALT_SHORTCUT_TXT);
   EXPECT_TRUE(file_util::Delete(prefs, false));
 }
-