[Page Load Metrics] PageLoadMetrics Mojofication.

chrome/browser/page_load_metrics/metrics_web_contents_observer.cc

Index: chrome/browser/page_load_metrics/metrics_web_contents_observer.cc
diff --git a/chrome/browser/page_load_metrics/metrics_web_contents_observer.cc b/chrome/browser/page_load_metrics/metrics_web_contents_observer.cc
index 044495e8c15477c6feb9d26f3b8e58b564fdebe0..531aae16f9aad52c1a3abbdb99c488a6f8029854 100644
--- a/chrome/browser/page_load_metrics/metrics_web_contents_observer.cc
+++ b/chrome/browser/page_load_metrics/metrics_web_contents_observer.cc
@@ -70,7 +70,8 @@ MetricsWebContentsObserver::MetricsWebContentsObserver(
     : content::WebContentsObserver(web_contents),
       in_foreground_(false),
       embedder_interface_(std::move(embedder_interface)),
-      has_navigated_(false) {
+      has_navigated_(false),
+      page_load_metrics_binding_(web_contents, this) {
   RegisterInputEventObserver(web_contents->GetRenderViewHost());
 }
 
@@ -610,6 +611,14 @@ void MetricsWebContentsObserver::OnTimingUpdated(
     observer.OnTimingUpdated(timing, metadata);
 }
 
+void MetricsWebContentsObserver::UpdateTiming(
+    const PageLoadTiming& timing,
+    const PageLoadMetadata& metadata) {
+  content::RenderFrameHost* render_frame_host =

[Bryan McQuade, 2017/05/07 19:48:13]

can we DCHECK that the mojofication feature is enabled here? and likewise,
dcheck that it's not enabled when the IPC codepath is hit? that probably means
adding an extra method besides OnTimingUpdated so you have an IPC-specific
method to dcheck in.

[lpy, 2017/05/08 18:38:33]

Done.

+      page_load_metrics_binding_.GetCurrentTargetFrame();
+  OnTimingUpdated(render_frame_host, timing, metadata);
+}
+
 bool MetricsWebContentsObserver::ShouldTrackNavigation(
     content::NavigationHandle* navigation_handle) const {
   DCHECK(navigation_handle->IsInMainFrame());