[PageLoadMetrics] Remove legacy IPC from Page Load Metrics.

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 ef05b3fef010fd2f3b972434853d26d22a19945d..b12f786b7d0b497856f25362384b958b42bf86f2 100644
--- a/chrome/browser/page_load_metrics/metrics_web_contents_observer.cc
+++ b/chrome/browser/page_load_metrics/metrics_web_contents_observer.cc
@@ -9,7 +9,6 @@
 #include <string>
 #include <utility>
 
-#include "base/feature_list.h"
 #include "base/location.h"
 #include "base/memory/ptr_util.h"
 #include "base/metrics/histogram_macros.h"
@@ -19,8 +18,6 @@
 #include "chrome/browser/page_load_metrics/page_load_metrics_util.h"
 #include "chrome/browser/page_load_metrics/page_load_tracker.h"
 #include "chrome/browser/prerender/prerender_contents.h"
-#include "chrome/common/chrome_features.h"
-#include "chrome/common/page_load_metrics/page_load_metrics_messages.h"
 #include "chrome/common/page_load_metrics/page_load_timing.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/global_request_id.h"
@@ -31,8 +28,6 @@
 #include "content/public/browser/web_contents.h"
 #include "content/public/browser/web_contents_observer.h"
 #include "content/public/browser/web_contents_user_data.h"
-#include "ipc/ipc_message.h"
-#include "ipc/ipc_message_macros.h"
 #include "net/base/net_errors.h"
 #include "ui/base/page_transition_types.h"
 
@@ -154,19 +149,6 @@ void MetricsWebContentsObserver::MediaStartedPlaying(
         video_type, render_frame_host == web_contents()->GetMainFrame());
 }
 
-bool MetricsWebContentsObserver::OnMessageReceived(
-    const IPC::Message& message,
-    content::RenderFrameHost* render_frame_host) {
-  DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
-  bool handled = true;
-  IPC_BEGIN_MESSAGE_MAP_WITH_PARAM(MetricsWebContentsObserver, message,
-                                   render_frame_host)
-    IPC_MESSAGE_HANDLER(PageLoadMetricsMsg_TimingUpdated, OnUpdateTimingOverIPC)
-    IPC_MESSAGE_UNHANDLED(handled = false)
-  IPC_END_MESSAGE_MAP()
-  return handled;
-}
-
 void MetricsWebContentsObserver::WillStartNavigationRequest(
     content::NavigationHandle* navigation_handle) {
   // Same-document navigations should never go through
@@ -622,27 +604,15 @@ void MetricsWebContentsObserver::OnTimingUpdated(
                                                               timing, metadata);
 }
 
-void MetricsWebContentsObserver::OnUpdateTimingOverIPC(
-    content::RenderFrameHost* render_frame_host,
-    const mojom::PageLoadTiming& timing,
-    const mojom::PageLoadMetadata& metadata) {
-  DCHECK(!base::FeatureList::IsEnabled(features::kPageLoadMetricsMojofication));
-  OnTimingUpdated(render_frame_host, timing, metadata);
-
-  for (auto& observer : testing_observers_)
-    observer.DidReceiveTimingUpdate(TestingObserver::IPCType::LEGACY);
-}
-
 void MetricsWebContentsObserver::UpdateTiming(
     const mojom::PageLoadTimingPtr timing,
     const mojom::PageLoadMetadataPtr metadata) {
   content::RenderFrameHost* render_frame_host =
       page_load_metrics_binding_.GetCurrentTargetFrame();
-  DCHECK(base::FeatureList::IsEnabled(features::kPageLoadMetricsMojofication));
   OnTimingUpdated(render_frame_host, *timing, *metadata);
 
   for (auto& observer : testing_observers_)
-    observer.DidReceiveTimingUpdate(TestingObserver::IPCType::MOJO);
+    observer.DidReceiveTimingUpdate();

[Bryan McQuade, 2017/06/23 15:15:07]

let's remove this

[lpy, 2017/06/23 21:23:46]

Done.

 }
 
 bool MetricsWebContentsObserver::ShouldTrackNavigation(