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

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

Issue 2737563007: Add support for tracking loading behavior of child frames. (Closed)
Patch Set: test fixes 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_tracker.h
diff --git a/chrome/browser/page_load_metrics/page_load_tracker.h b/chrome/browser/page_load_metrics/page_load_tracker.h
index c08628d0d7236c1f13edb318241a19fee8fc8e6f..4f2c90b74d6d763a2aaffa313f6db3bdbfcde1ca 100644
--- a/chrome/browser/page_load_metrics/page_load_tracker.h
+++ b/chrome/browser/page_load_metrics/page_load_tracker.h
@@ -107,6 +107,9 @@ enum InternalErrorLoadEvent {
// No page load end time was recorded for this page load.
ERR_NO_PAGE_LOAD_END_TIME,
+ // Received a timing update from a subframe.
+ ERR_TIMING_IPC_FROM_SUBFRAME,
+
// Add values before this final count.
ERR_LAST_ENTRY,
};
@@ -160,6 +163,8 @@ class PageLoadTracker {
bool UpdateTiming(const PageLoadTiming& timing,
const PageLoadMetadata& metadata);
+ void UpdateChildMetadata(const PageLoadMetadata& child_metadata);
+
void OnLoadedResource(const ExtraRequestInfo& extra_request_info);
// Signals that we should stop tracking metrics for the associated page load.
@@ -303,6 +308,9 @@ class PageLoadTracker {
// Whether this page load was user initiated.
UserInitiatedInfo user_initiated_info_;
+ // WebLoadingBehaviorFlags for all child frames in this page.
+ int child_loading_behavior_flags_;
+
// This is a subtle member. If a provisional load A gets aborted by
// provisional load B, which gets aborted by C that eventually commits, then
// there exists an abort chain of length 2, starting at A's navigation_start.

Powered by Google App Engine
This is Rietveld 408576698