Replace IPC messages in layout_test_render_frame_observer.

content/shell/renderer/layout_test/blink_test_runner.h

Index: content/shell/renderer/layout_test/blink_test_runner.h
diff --git a/content/shell/renderer/layout_test/blink_test_runner.h b/content/shell/renderer/layout_test/blink_test_runner.h
index 4d69a11e51243d78c4993ce506c0cc0174c6602f..db45a7cbf399a57823442de76c10628305e92692 100644
--- a/content/shell/renderer/layout_test/blink_test_runner.h
+++ b/content/shell/renderer/layout_test/blink_test_runner.h
@@ -17,8 +17,8 @@
 #include "content/public/common/page_state.h"
 #include "content/public/renderer/render_view_observer.h"
 #include "content/public/renderer/render_view_observer_tracker.h"
+#include "content/shell/common/layout_test.mojom.h"
 #include "content/shell/common/layout_test/layout_test_bluetooth_fake_adapter_setter.mojom.h"
-#include "content/shell/common/shell_test_configuration.h"
 #include "v8/include/v8.h"
 
 class SkBitmap;
@@ -173,8 +173,9 @@ class BlinkTestRunner : public RenderViewObserver,
   void ReportLeakDetectionResult(const LeakDetectionResult& result);
 
   // Message handlers forwarded by LayoutTestRenderFrameObserver.
-  void OnSetTestConfiguration(const ShellTestConfiguration& params);
-  void OnReplicateTestConfiguration(const ShellTestConfiguration& params);
+  void OnSetTestConfiguration(const mojom::ShellTestConfigurationPtr& params);

[Sam McNally, 2017/01/26 23:24:08]

Pass these by value.

[Jia, 2017/01/29 23:14:35]

Done. I used const ref to avoid Clone in some operations, but if it's better to
avoid using pass-by-ref and if clone isn't too expensive (I guess it's
inexpensive here), then I'll use pass-by-value.

+  void OnReplicateTestConfiguration(
+      const mojom::ShellTestConfigurationPtr& params);
   void OnSetupSecondaryRenderer();
 
  private:
@@ -207,7 +208,7 @@ class BlinkTestRunner : public RenderViewObserver,
 
   test_runner::TestPreferences prefs_;
 
-  ShellTestConfiguration test_config_;
+  mojom::ShellTestConfigurationPtr test_config_;
 
   std::vector<int> routing_ids_;
   std::vector<std::vector<PageState> > session_histories_;