bluetooth: Don't call methods of WebContentsImpl while it's being destroyed.

content/test/test_render_frame_host.h

Index: content/test/test_render_frame_host.h
diff --git a/content/test/test_render_frame_host.h b/content/test/test_render_frame_host.h
index f1718467fb7fcff9d4f4834464261bbe99f1c29a..9789f5846052b3283b021ded569ee432fa3f4d01 100644
--- a/content/test/test_render_frame_host.h
+++ b/content/test/test_render_frame_host.h
@@ -10,6 +10,7 @@
 #include <vector>
 
 #include "base/macros.h"
+#include "content/browser/bluetooth/frame_connected_bluetooth_devices.h"

[Jeffrey Yasskin, 2016/06/06 20:32:20]

You could get by with just a forward declaration in the header, since the
destructor and the unique_ptr<> use are in the .cc file.

[ortuno, 2016/06/06 22:38:05]

Done.

 #include "content/browser/frame_host/render_frame_host_impl.h"
 #include "content/public/browser/web_contents_observer.h"
 #include "content/public/test/mock_render_process_host.h"
@@ -107,6 +108,9 @@ class TestRenderFrameHost : public RenderFrameHostImpl,
   void SendRendererInitiatedNavigationRequest(const GURL& url,
                                               bool has_user_gesture);
 
+  void SetFrameConnectedBluetoothDevices(
+      std::unique_ptr<FrameConnectedBluetoothDevices> connected_devices);
+
   void DidChangeOpener(int opener_routing_id);
 
   void DidEnforceStrictMixedContentChecking();
@@ -153,6 +157,8 @@ class TestRenderFrameHost : public RenderFrameHostImpl,
 
   std::string contents_mime_type_;
 
+  std::unique_ptr<FrameConnectedBluetoothDevices> connected_devices_;

[Jeffrey Yasskin, 2016/06/06 20:32:20]

Maybe comment that this exists to test destruction of an instance of this type?
I'm not sure that the owners will want that comment, but it's a somewhat unusual
reason to have a member here.

[ortuno, 2016/06/06 22:38:05]

Done.

+
   // See set_simulate_history_list_was_cleared() above.
   bool simulate_history_list_was_cleared_;