Record metrics for slow startups

chrome/browser/chrome_browser_main.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/chrome_browser_main.h"
 
 #if defined(TOOLKIT_GTK)
 #include <gtk/gtk.h>
 #endif
 
@@ skipping 1598 matching lines @@
 // Windows.
 #if defined(OS_MACOSX) || defined(OS_WIN)
   const base::Time *process_creation_time =
       base::CurrentProcessInfo::CreationTime();
 
   if (process_creation_time)
     RecordPreReadExperimentTime("Startup.BrowserMessageLoopStartTime",
         base::Time::Now() - *process_creation_time);
 #endif // OS_MACOSX || OS_WIN
 
-  // Startup.BrowserMessageLoopStartTime exhibits instability in the field
-  // which limits its usefullness in all scenarios except when we have a very
-  // large sample size.
-  // Attempt to mitigate this with a new metric:
-  // * Measure time from main entry rather than the OS' notion of process start
-  //   time.
-  // * Only measure launches that occur 7 minutes after boot to try to avoid
-  //   cases where Chrome is auto-started and IO is heavily loaded.
-  const int64 kSevenMinutesInMilliseconds =
-      base::TimeDelta::FromMinutes(7).InMilliseconds();
-  if (base::SysInfo::Uptime() < kSevenMinutesInMilliseconds)
-    return;
-
-  // Set up to match Startup.BrowserMessageLoopStartTime measurement above.
-  const base::TimeDelta kStartupTimeMin(base::TimeDelta::FromMilliseconds(1));
-  const base::TimeDelta kStartupTimeMax(base::TimeDelta::FromHours(1));
-  static const size_t kStartupTimeBuckets(100);
-  HISTOGRAM_CUSTOM_TIMES(
-      "Startup.BrowserMessageLoopStartTimeFromMainEntry",
-      base::Time::Now() - startup_metric_utils::MainEntryStartTime(),
-      kStartupTimeMin,
-      kStartupTimeMax,
-      kStartupTimeBuckets);
+  // Record collected startup metrics.
+  startup_metric_utils::OnBrowserStartupComplete();
 }
 
 // This code is specific to the Windows-only PreReadExperiment field-trial.
 void RecordPreReadExperimentTime(const char* name, base::TimeDelta time) {
   DCHECK(name != NULL);
 
   // This gets called with different histogram names, so we don't want to use
   // the UMA_HISTOGRAM_CUSTOM_TIMES macro--it uses a static variable, and the
   // first call wins.
   AddPreReadHistogramTime(name, time);
@@ skipping 14 matching lines @@
     if (base::win::GetVersion() <= base::win::VERSION_XP)
       uma_name += "_XP";
 
     uma_name += "_PreRead_";
     uma_name += pre_read_percentage;
     AddPreReadHistogramTime(uma_name.c_str(), time);
   }
 #endif
 #endif
 }