bluetooth: Create Mojo equivalent of SetBluetoothMockDataSet

content/shell/renderer/layout_test/blink_test_runner.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/shell/renderer/layout_test/blink_test_runner.h"
 
 #include <stddef.h>
 #include <algorithm>
 #include <clocale>
 #include <cmath>
@@ skipping 16 matching lines @@
 #include "base/thread_task_runner_handle.h"
 #include "base/time/time.h"
 #include "base/values.h"
 #include "build/build_config.h"
 #include "components/plugins/renderer/plugin_placeholder.h"
 #include "components/test_runner/gamepad_controller.h"
 #include "components/test_runner/web_test_interfaces.h"
 #include "components/test_runner/web_test_proxy.h"
 #include "components/test_runner/web_test_runner.h"
 #include "content/public/common/content_switches.h"
+#include "content/public/common/service_registry.h"
 #include "content/public/common/url_constants.h"
 #include "content/public/common/web_preferences.h"
 #include "content/public/renderer/media_stream_utils.h"
 #include "content/public/renderer/render_frame.h"
 #include "content/public/renderer/render_view.h"
 #include "content/public/renderer/render_view_visitor.h"
 #include "content/public/renderer/renderer_gamepad_provider.h"
 #include "content/public/test/layouttest_support.h"
 #include "content/shell/common/layout_test/layout_test_messages.h"
 #include "content/shell/common/shell_messages.h"
@@ skipping 425 matching lines @@
 
 void BlinkTestRunner::EnableUseZoomForDSF() {
   base::CommandLine::ForCurrentProcess()->
       AppendSwitch(switches::kEnableUseZoomForDSF);
 }
 
 void BlinkTestRunner::SetDeviceColorProfile(const std::string& name) {
   content::SetDeviceColorProfile(render_view(), name);
 }
 
-void BlinkTestRunner::SetBluetoothMockDataSet(const std::string& name) {
-  Send(new LayoutTestHostMsg_SetBluetoothAdapter(name));
-  // Auto-reset the chooser type so we don't get order dependence when some
-  // tests forget to do it explicitly.
-  Send(new ShellViewHostMsg_SetBluetoothManualChooser(routing_id(), false));
+void BlinkTestRunner::SetBluetoothFakeAdapter(const std::string& adapter_name,
+                                              const base::Closure& callback) {
+  GetBluetoothFakeAdapterSetter()->Set(adapter_name, callback);
 }
 
-void BlinkTestRunner::SetBluetoothManualChooser() {
-  Send(new ShellViewHostMsg_SetBluetoothManualChooser(routing_id(), true));
+void BlinkTestRunner::SetBluetoothManualChooser(bool enable) {
+  Send(new ShellViewHostMsg_SetBluetoothManualChooser(routing_id(), enable));
 }
 
 void BlinkTestRunner::GetBluetoothManualChooserEvents(
     const base::Callback<void(const std::vector<std::string>&)>& callback) {
   get_bluetooth_events_callbacks_.push_back(callback);
   Send(new ShellViewHostMsg_GetBluetoothManualChooserEvents(routing_id()));
 }
 
 void BlinkTestRunner::SendBluetoothManualChooserEvent(
     const std::string& event,
@@ skipping 437 matching lines @@
 
 void BlinkTestRunner::CaptureDumpComplete() {
   render_view()->GetWebView()->mainFrame()->stopLoading();
 
   base::ThreadTaskRunnerHandle::Get()->PostTask(
       FROM_HERE, base::Bind(base::IgnoreResult(&BlinkTestRunner::Send),
                             base::Unretained(this),
                             new ShellViewHostMsg_TestFinished(routing_id())));
 }
 
+mojom::LayoutTestBluetoothFakeAdapterSetterPtr&
+BlinkTestRunner::GetBluetoothFakeAdapterSetter() {
+  if (!bluetooth_fake_adapter_setter_) {
+    RenderThread::Get()->GetServiceRegistry()->ConnectToRemoteService(
+        mojo::GetProxy(&bluetooth_fake_adapter_setter_));
+  }
+  return bluetooth_fake_adapter_setter_;

[Ken Rockot(use gerrit already), 2016/03/29 20:53:23]

nit: I think it's weird to return a reference to a scoped_ptr instead of simply
returning a raw pointer. You can return bluetooth_fake_adapter_setter_.get(),
which is of type mojom::LayoutTestBluetoothFakeAdapterSetter*. It will work the
same, with the added bonus that the caller can't inadvertently trash the
InterfacePtr.

[ortuno, 2016/03/29 21:17:04]

Ha. Makes sense. I saw VR doing the same so I just copied it. Changed to
pointer.

[Jeffrey Yasskin, 2016/03/29 22:19:10]

Or, since this can never return null, return the
mojom::LayoutTestBluetoothFakeAdapterSetter& *bluetooth_fake_adapter_setter_?

[ortuno, 2016/03/29 22:30:53]

Done.

+}
+
 void BlinkTestRunner::OnReplicateTestConfiguration(
     const ShellTestConfiguration& params) {
   test_config_ = params;
   is_main_window_ = true;
 
   test_runner::WebTestInterfaces* interfaces =
       LayoutTestRenderProcessObserver::GetInstance()->test_interfaces();
   interfaces->SetTestIsRunning(true);
   interfaces->ConfigureForTestWithURL(params.test_url,
                                       params.enable_pixel_dumping);
@@ skipping 51 matching lines @@
   get_bluetooth_events_callbacks_.pop_front();
   callback.Run(events);
 }
 
 void BlinkTestRunner::ReportLeakDetectionResult(
     const LeakDetectionResult& report) {
   Send(new ShellViewHostMsg_LeakDetectionDone(routing_id(), report));
 }
 
 }  // namespace content