Refactor PageLoadMetricsObserver completion callback policy

chrome/browser/page_load_metrics/observers/core_page_load_metrics_observer.h

Index: chrome/browser/page_load_metrics/observers/core_page_load_metrics_observer.h
diff --git a/chrome/browser/page_load_metrics/observers/core_page_load_metrics_observer.h b/chrome/browser/page_load_metrics/observers/core_page_load_metrics_observer.h
index ba281a70002fc495677a63447581ffd607fb2865..ad86d75dbf83d92df7c3cf706ace2a3a0c2d92b5 100644
--- a/chrome/browser/page_load_metrics/observers/core_page_load_metrics_observer.h
+++ b/chrome/browser/page_load_metrics/observers/core_page_load_metrics_observer.h
@@ -88,25 +88,15 @@ class CorePageLoadMetricsObserver
   void OnComplete(const page_load_metrics::PageLoadTiming& timing,
                   const page_load_metrics::PageLoadExtraInfo& info) override;
   void OnFailedProvisionalLoad(
-      content::NavigationHandle* navigation_handle) override;
+      const page_load_metrics::FailedProvisionalLoadInfo& failed_load_info,
+      const page_load_metrics::PageLoadExtraInfo& extra_info) override;
 
  private:
-  // Information related to failed provisional loads.
-  // Populated in OnFailedProvisionalLoad and accessed in OnComplete.
-  struct FailedProvisionalLoadInfo {
-    base::Optional<base::TimeDelta> interval;
-    net::Error error;
-
-    FailedProvisionalLoadInfo();
-    ~FailedProvisionalLoadInfo();
-  };
-
   void RecordTimingHistograms(const page_load_metrics::PageLoadTiming& timing,
                               const page_load_metrics::PageLoadExtraInfo& info);
   void RecordRappor(const page_load_metrics::PageLoadTiming& timing,
                     const page_load_metrics::PageLoadExtraInfo& info);
 
-  FailedProvisionalLoadInfo failed_provisional_load_info_;
   ui::PageTransition transition_;
   bool initiated_by_user_gesture_;