Replace IPC messages in layout_test_render_frame_observer.

content/shell/renderer/layout_test/layout_test_render_frame_observer.cc

Index: content/shell/renderer/layout_test/layout_test_render_frame_observer.cc
diff --git a/content/shell/renderer/layout_test/layout_test_render_frame_observer.cc b/content/shell/renderer/layout_test/layout_test_render_frame_observer.cc
index 4417b57ffa342cc971e3e0c685c4ecbec85b1c95..3ccb7751d3f95394fb72bf2ebe58568f904613c1 100644
--- a/content/shell/renderer/layout_test/layout_test_render_frame_observer.cc
+++ b/content/shell/renderer/layout_test/layout_test_render_frame_observer.cc
@@ -8,6 +8,7 @@
 
 #include "components/test_runner/web_test_interfaces.h"
 #include "components/test_runner/web_test_runner.h"
+#include "content/public/common/associated_interface_registry.h"
 #include "content/public/renderer/render_frame.h"
 #include "content/shell/common/shell_messages.h"
 #include "content/shell/renderer/layout_test/blink_test_runner.h"
@@ -19,36 +20,29 @@ namespace content {
 
 LayoutTestRenderFrameObserver::LayoutTestRenderFrameObserver(
     RenderFrame* render_frame)
-    : RenderFrameObserver(render_frame) {
+    : RenderFrameObserver(render_frame), binding_(this) {
   render_frame->GetWebFrame()->setContentSettingsClient(
       LayoutTestRenderThreadObserver::GetInstance()
           ->test_interfaces()
           ->TestRunner()
-          ->GetWebContentSettings());
+      ->GetWebContentSettings());
+  render_frame->GetAssociatedInterfaceRegistry()->AddInterface(
+      base::Bind(&LayoutTestRenderFrameObserver::BindRequest,
+                 base::Unretained(this)));
 }
 
-bool LayoutTestRenderFrameObserver::OnMessageReceived(
-    const IPC::Message& message) {
-  bool handled = true;
-  IPC_BEGIN_MESSAGE_MAP(LayoutTestRenderFrameObserver, message)
-    IPC_MESSAGE_HANDLER(ShellViewMsg_LayoutDumpRequest, OnLayoutDumpRequest)
-    IPC_MESSAGE_HANDLER(ShellViewMsg_ReplicateTestConfiguration,
-                        OnReplicateTestConfiguration)
-    IPC_MESSAGE_HANDLER(ShellViewMsg_SetTestConfiguration,
-                        OnSetTestConfiguration)
-    IPC_MESSAGE_HANDLER(ShellViewMsg_SetupSecondaryRenderer,
-                        OnSetupSecondaryRenderer)
-    IPC_MESSAGE_UNHANDLED(handled = false)
-  IPC_END_MESSAGE_MAP()
+LayoutTestRenderFrameObserver::~LayoutTestRenderFrameObserver() {}

[tibell, 2017/01/19 00:29:54]

Nit: I believe

LayoutTestRenderFrameObserver::~LayoutTestRenderFrameObserver() = default;

works.

[Jia, 2017/01/20 01:52:15]

Done.

 
-  return handled;
+void LayoutTestRenderFrameObserver::BindRequest(
+    mojom::ShellViewAssociatedRequest request) {
+  binding_.Bind(std::move(request));
 }
 
 void LayoutTestRenderFrameObserver::OnDestruct() {
   delete this;
 }
 
-void LayoutTestRenderFrameObserver::OnLayoutDumpRequest() {
+void LayoutTestRenderFrameObserver::LayoutDumpRequest() {
   std::string dump =
       LayoutTestRenderThreadObserver::GetInstance()
           ->test_interfaces()
@@ -57,19 +51,19 @@ void LayoutTestRenderFrameObserver::OnLayoutDumpRequest() {
   Send(new ShellViewHostMsg_LayoutDumpResponse(routing_id(), dump));
 }
 
-void LayoutTestRenderFrameObserver::OnReplicateTestConfiguration(
+void LayoutTestRenderFrameObserver::ReplicateTestConfiguration(
     const ShellTestConfiguration& test_config) {
   BlinkTestRunner::Get(render_frame()->GetRenderView())
       ->OnReplicateTestConfiguration(test_config);
 }
 
-void LayoutTestRenderFrameObserver::OnSetTestConfiguration(
+void LayoutTestRenderFrameObserver::SetTestConfiguration(
     const ShellTestConfiguration& test_config) {
   BlinkTestRunner::Get(render_frame()->GetRenderView())
       ->OnSetTestConfiguration(test_config);
 }
 
-void LayoutTestRenderFrameObserver::OnSetupSecondaryRenderer() {
+void LayoutTestRenderFrameObserver::SetupSecondaryRenderer() {
   BlinkTestRunner::Get(render_frame()->GetRenderView())
       ->OnSetupSecondaryRenderer();
 }