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

Unified Diff: components/startup_metric_utils/startup_metric_utils.h

Issue 1407093003: Add start/finish navigation to startup metrics. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@a2_better_ownership_model_firstwebcontentsprofiler
Patch Set: Using DidFinishNavigation() 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 f2d13264823b4ce59c314249bc73804c527cdc8c..54a66cac7f4d0d1ceeb42188bc9d296f704ca606 100644
--- a/components/startup_metric_utils/startup_metric_utils.h
+++ b/components/startup_metric_utils/startup_metric_utils.h
@@ -74,6 +74,18 @@ void RecordFirstWebContentsMainFrameLoad(const base::Time& time);
// Call this with the time when the first web contents had a non-empty paint.
void RecordFirstWebContentsNonEmptyPaint(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);
+
+// Call this with the time when the first web contents completed a navigation
+// for its main frame.
+void RecordFirstWebContentsMainNavigationFinishedFirst(const base::Time& time);
+
+// Call this with the time when the first web contents completed all navigations
+// for its main frame.
+void RecordFirstWebContentsMainNavigationFinishedAll(const base::Time& time);
+
// Returns the time of main entry recorded from RecordMainEntryPointTime.
// Returns a null Time if a value has not been recorded yet.
// This method is expected to be called from the UI thread.

Powered by Google App Engine
This is Rietveld 408576698