Index: components/startup_metric_utils/browser/startup_metric_utils.h |
diff --git a/components/startup_metric_utils/browser/startup_metric_utils.h b/components/startup_metric_utils/browser/startup_metric_utils.h |
index ba6c9cfff144d2821ab243df2956f570b2ef5268..cdfb55c99ce50954e40baff65a3183f6f4123a6d 100644 |
--- a/components/startup_metric_utils/browser/startup_metric_utils.h |
+++ b/components/startup_metric_utils/browser/startup_metric_utils.h |
@@ -34,7 +34,8 @@ enum StartupTemperature { |
// The startup type couldn't quite be classified as warm or cold, but rather |
// was somewhere in between. |
LUKEWARM_STARTUP_TEMPERATURE = 2, |
- // This must be after all meaningful values. |
+ // This must be after all meaningful values. All new values should be added |
+ // above this one. |
STARTUP_TEMPERATURE_COUNT, |
// Startup temperature wasn't yet determined. |
UNDETERMINED_STARTUP_TEMPERATURE |
@@ -76,21 +77,13 @@ void RecordMainEntryPointTime(const base::Time& time); |
void RecordExeMainEntryPointTime(const base::Time& time); |
// Call this with the time recorded just before the message loop is started. |
-// |is_first_run| - is the current launch part of a first run. |
+// |is_first_run| - is the current launch part of a first run. |pref_service| is |
+// an optional parameter which, if provided, will be used to store state for |
+// stats that span multiple startups; in its absence those stats will not be |
+// recorded. |
void RecordBrowserMainMessageLoopStart(const base::TimeTicks& ticks, |
- bool is_first_run); |
- |
-// Logs the Startup.TimeSinceLastStartup histogram. Obtains the timestamp of the |
-// last startup from |pref_service| and overwrites it with the timestamp of the |
-// current startup. If the startup temperature has been set by |
-// RecordBrowserMainMessageLoopStart, the time since last startup is also logged |
-// to an histogram suffixed with the startup temperature. |
-void RecordTimeSinceLastStartup(PrefService* pref_service); |
- |
-// Logs the Startup.SameVersionStartupCount histogram. Relies on |pref_service| |
-// to know information about the previous startups and store information for |
-// future ones. |
-void RecordStartupCount(PrefService* pref_service); |
+ bool is_first_run, |
+ PrefService* pref_service); |
// Call this with the time when the first browser window became visible. |
void RecordBrowserWindowDisplay(const base::TimeTicks& ticks); |