[page_load_metrics] User Initiated Abort Tracking (Observer version)

chrome/browser/page_load_metrics/observers/stale_while_revalidate_metrics_observer.cc

Index: chrome/browser/page_load_metrics/observers/stale_while_revalidate_metrics_observer.cc
diff --git a/chrome/browser/page_load_metrics/observers/stale_while_revalidate_metrics_observer.cc b/chrome/browser/page_load_metrics/observers/stale_while_revalidate_metrics_observer.cc
index 58df2d123978ae925e7a32dd63b33348bd841b10..e0ec33f7552ed1c2948d4bd92d2ef10503a01fbf 100644
--- a/chrome/browser/page_load_metrics/observers/stale_while_revalidate_metrics_observer.cc
+++ b/chrome/browser/page_load_metrics/observers/stale_while_revalidate_metrics_observer.cc
@@ -23,22 +23,30 @@ void StaleWhileRevalidateMetricsObserver::OnCommit(
 void StaleWhileRevalidateMetricsObserver::OnComplete(
     const page_load_metrics::PageLoadTiming& timing,
     const page_load_metrics::PageLoadExtraInfo& extra_info) {
+  using page_load_metrics::EventOccurredInForeground;
+
   if (!is_interesting_domain_)
     return;
 
-  if (!timing.load_event_start.is_zero()) {
+  // A page load that started in the background usually means a prerender. The
+  // performance metrics for a prerender aren't directly comparable to normal
+  // foreground page loads, so just drop them.
+  if (!extra_info.started_in_foreground)
+    return;
+
+  if (EventOccurredInForeground(timing.load_event_start, extra_info)) {

[Bryan McQuade, 2015/12/09 20:35:23]

I agree that we should do this, but can you make sure adam is aware of this
change?

[Charlie Harrison, 2015/12/09 22:46:43]

Sorry this shouldn't be included in this change.

     PAGE_LOAD_HISTOGRAM(
         "PageLoad.Clients.StaleWhileRevalidateExperiment.Timing2."
         "NavigationToLoadEventFired",
         timing.load_event_start);
   }
-  if (!timing.first_layout.is_zero()) {
+  if (EventOccurredInForeground(timing.first_layout, extra_info)) {
     PAGE_LOAD_HISTOGRAM(
         "PageLoad.Clients.StaleWhileRevalidateExperiment.Timing2."
         "NavigationToFirstLayout",
         timing.first_layout);
   }
-  if (!timing.first_text_paint.is_zero()) {
+  if (EventOccurredInForeground(timing.first_text_paint, extra_info)) {
     PAGE_LOAD_HISTOGRAM(
         "PageLoad.Clients.StaleWhileRevalidateExperiment.Timing2."
         "NavigationToFirstTextPaint",