Add SameVersionStartupCounts suffix to startup HardFault and Temperature histograms.

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"
 
 #include <stddef.h>
 #include <stdint.h>
 #include <set>
 #include <string>
@@ skipping 774 matching lines @@
   bool is_first_run = false;
 #if !defined(OS_ANDROID)
   // On Android, first run is handled in Java code, and the C++ side of Chrome
   // doesn't know if this is the first run. This will cause some inaccuracy in
   // the UMA statistics, but this should be minor (first runs are rare).
   is_first_run = first_run::IsChromeFirstRun();
 #endif  // defined(OS_ANDROID)
 
   // Record collected startup metrics.
   startup_metric_utils::RecordBrowserMainMessageLoopStart(
-      base::TimeTicks::Now(), is_first_run);
-  startup_metric_utils::RecordTimeSinceLastStartup(
-      g_browser_process->local_state());
-  startup_metric_utils::RecordStartupCount(g_browser_process->local_state());
+      base::TimeTicks::Now(), is_first_run, g_browser_process->local_state());

[fdoray, 2016/01/25 18:48:00]

You will need to update this call site:
https://code.google.com/p/chromium/codesearch#chromium/src/components/html_vi...

[gab, 2016/01/25 21:20:16]

Argh.. @msw: is there an equivalent to local_state in html_viewer (i.e. a
persistent PrefService)? Or should we split what needs it into a separate public
call and not call it from here?

[msw, 2016/01/25 21:29:44]

No, not afaik, but you can just pass a null base::PrefService in html_viewer and
handle that case within RecordBrowserMainMessageLoopStart, right?

[gab, 2016/01/25 21:32:26]

Was hoping not to add special code on our side for html_viewer, but yes..

 }
 
 // -----------------------------------------------------------------------------
 // TODO(viettrungluu): move more/rest of BrowserMain() into BrowserMainParts.
 
 #if defined(OS_WIN)
 #define DLLEXPORT __declspec(dllexport)
 
 // We use extern C for the prototype DLLEXPORT to avoid C++ name mangling.
 extern "C" {
@@ skipping 1068 matching lines @@
   chromeos::CrosSettings::Shutdown();
 #endif  // defined(OS_CHROMEOS)
 #endif  // defined(OS_ANDROID)
 }
 
 // Public members:
 
 void ChromeBrowserMainParts::AddParts(ChromeBrowserMainExtraParts* parts) {
   chrome_extra_parts_.push_back(parts);
 }