commit ff5ce2374012237cdf8e2c3ef24908b1a13106b7
author siggi@chromium.org <siggi@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> Wed Dec 21 02:00:59 2011
committer siggi@chromium.org <siggi@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> Wed Dec 21 02:00:59 2011
tree 05fd5b123257416a0e118aba9558f510f17211a6
parent 9d02cda7a634fbd6e53d98091f618057f0174387

Make Startup histogram names unique on XP.

BUG=None
TEST=None


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

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

chrome/browser/chrome_browser_main.cc

diff --git a/chrome/browser/chrome_browser_main.cc b/chrome/browser/chrome_browser_main.cc
index d1b0bf33..47721c2 100644
--- a/chrome/browser/chrome_browser_main.cc
+++ b/chrome/browser/chrome_browser_main.cc
@@ -1986,6 +1986,12 @@
   if (env->GetVar(chrome::kPreReadEnvironmentVariable, &pre_read) &&
       (pre_read == "0" || pre_read == "1")) {
     std::string uma_name(name);
+
+    // We want XP to record a separate histogram, as the loader on XP
+    // is very different from the Vista and Win7 loaders.
+    if (base::win::GetVersion() <= base::win::VERSION_XP)
+      uma_name += "_XP";
+
     uma_name += "_PreRead";
     uma_name += pre_read == "1" ? "Enabled" : "Disabled";
     AddPreReadHistogramTime(uma_name.c_str(), time);