Better way to implement FrameConnectedBluetoothDevicesTest

content/test/test_render_frame_host.cc

Index: content/test/test_render_frame_host.cc
diff --git a/content/test/test_render_frame_host.cc b/content/test/test_render_frame_host.cc
index 8a21ccfd4ec6d2020c0bfc048a6af225b41b030e..88fd540ffc609c87b76451fe143d06e7338f056a 100644
--- a/content/test/test_render_frame_host.cc
+++ b/content/test/test_render_frame_host.cc
@@ -20,7 +20,9 @@
 #include "content/test/browser_side_navigation_test_utils.h"
 #include "content/test/test_navigation_url_loader.h"
 #include "content/test/test_render_view_host.h"
+#include "mojo/public/cpp/bindings/interface_request.h"
 #include "net/base/load_flags.h"
+#include "third_party/WebKit/public/platform/modules/bluetooth/web_bluetooth.mojom.h"
 #include "third_party/WebKit/public/platform/WebPageVisibilityState.h"
 #include "third_party/WebKit/public/web/WebSandboxFlags.h"
 #include "third_party/WebKit/public/web/WebTreeScopeType.h"
@@ -428,6 +430,19 @@ void TestRenderFrameHost::PrepareForCommitWithServerRedirect(
   url_loader->CallOnResponseStarted(response, MakeEmptyStream(), nullptr);
 }
 
+WebBluetoothServiceImpl*
+TestRenderFrameHost::CreateWebBluetoothServiceForTesting() {
+  // TODO(nick): Right now I'm leaking the WebBluetoothServicePtr, because I
+  // don't know the lifetime implications. We should either free it here or
+  // return it to the caller. In fact, it may be desireable to return it to the
+  // caller anyway (as a pair<ServicePtr, WebBluetoothServiceImpl> ?)
+  auto service_ptr = new blink::mojom::WebBluetoothServicePtr();
+  WebBluetoothServiceImpl* service =
+      RenderFrameHostImpl::CreateWebBluetoothService(
+          mojo::GetProxy(service_ptr));  // TODO(nick): Don't leak service_ptr.
+  return service;
+}
+
 int32_t TestRenderFrameHost::ComputeNextPageID() {
   const NavigationEntryImpl* entry = static_cast<NavigationEntryImpl*>(
       frame_tree_node()->navigator()->GetController()->GetPendingEntry());