Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(299)

Unified Diff: components/startup_metric_utils/startup_metric_utils.h

Issue 1421293005: Move new Startup.FirstWebContents.* stats to Startup.FirstWebContents.*2. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@ab2_moar_betta_metrics
Patch Set: Created 5 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: components/startup_metric_utils/startup_metric_utils.h
diff --git a/components/startup_metric_utils/startup_metric_utils.h b/components/startup_metric_utils/startup_metric_utils.h
index d5b0e8fa19d226042a591ee12efa41d8c5041102..684046cb435a2305d48f6195fc19f3a3a0d3f095 100644
--- a/components/startup_metric_utils/startup_metric_utils.h
+++ b/components/startup_metric_utils/startup_metric_utils.h
@@ -68,12 +68,22 @@ void RecordBrowserWindowDisplay(const base::Time& time);
// Call this with the time delta that the browser spent opening its tabs.
void RecordBrowserOpenTabsDelta(const base::TimeDelta& delta);
-// Call this with the time when the first web contents loaded its main frame.
+// Call this with the time when the first web contents loaded its main frame,
+// only if the first web contents was unimpended in its attempt to do so.
void RecordFirstWebContentsMainFrameLoad(const base::Time& time);
-// Call this with the time when the first web contents had a non-empty paint.
+// Call this with the time when the first web contents loaded its main frame.
+// This records an old stat kept for comparison purposes until M49.
+void RecordDeprecatedFirstWebContentsMainFrameLoad(const base::Time& time);
+
+// Call this with the time when the first web contents had a non-empty paint,
+// only if the first web contents was unimpended in its attempt to do so.
void RecordFirstWebContentsNonEmptyPaint(const base::Time& time);
+// Call this with the time when the first web contents had a non-empty paint.
+// This records an old stat kept for comparison purposes until M49.
+void RecordDeprecatedFirstWebContentsNonEmptyPaint(const base::Time& time);
+
// Call this with the time when the first web contents began navigating its main
// frame.
void RecordFirstWebContentsMainNavigationStart(const base::Time& time);
« no previous file with comments | « components/metrics/proto/profiler_event.proto ('k') | components/startup_metric_utils/startup_metric_utils.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698