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

Unified Diff: chrome/browser/page_load_metrics/page_load_metrics_observer.h

Issue 2737563007: Add support for tracking loading behavior of child frames. (Closed)
Patch Set: improve test comments Created 3 years, 9 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: chrome/browser/page_load_metrics/page_load_metrics_observer.h
diff --git a/chrome/browser/page_load_metrics/page_load_metrics_observer.h b/chrome/browser/page_load_metrics/page_load_metrics_observer.h
index 370d95c2dff214c69c641a9dc73ab3eb713ca155..80600dbb784babe703b344c0287c853d06843ed0 100644
--- a/chrome/browser/page_load_metrics/page_load_metrics_observer.h
+++ b/chrome/browser/page_load_metrics/page_load_metrics_observer.h
@@ -118,7 +118,8 @@ struct PageLoadExtraInfo {
PageEndReason page_end_reason,
UserInitiatedInfo page_end_user_initiated_info,
const base::Optional<base::TimeDelta>& page_end_time,
- const PageLoadMetadata& metadata);
+ const PageLoadMetadata& main_frame_metadata,
+ const PageLoadMetadata& child_frame_metadata);
// Simplified version of the constructor, intended for use in tests.
static PageLoadExtraInfo CreateForTesting(const GURL& url,
@@ -186,8 +187,11 @@ struct PageLoadExtraInfo {
const base::Optional<base::TimeDelta> page_end_time;
// Extra information supplied to the page load metrics system from the
- // renderer.
- const PageLoadMetadata metadata;
+ // renderer for the main frame.
+ const PageLoadMetadata main_frame_metadata;
+
+ // PageLoadMetadata for child frames of the current page load.
+ const PageLoadMetadata child_frame_metadata;
};
// Container for various information about a request within a page load.
@@ -314,7 +318,8 @@ class PageLoadMetricsObserver {
virtual void OnParseStop(const PageLoadTiming& timing,
const PageLoadExtraInfo& extra_info) {}
- // Invoked when there is a change in PageLoadMetadata's behavior_flags.
+ // Invoked when there is a change in either the main_frame_metadata or the
+ // child_frame_metadata's loading behavior_flags.
virtual void OnLoadingBehaviorObserved(
const page_load_metrics::PageLoadExtraInfo& extra_info) {}

Powered by Google App Engine
This is Rietveld 408576698