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

content/test/test_render_frame_host.h

 // Copyright 2013 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.
 
 #ifndef CONTENT_TEST_TEST_RENDER_FRAME_HOST_H_
 #define CONTENT_TEST_TEST_RENDER_FRAME_HOST_H_
 
 #include <stdint.h>
 
 #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"
 #include "content/public/test/test_renderer_host.h"
 #include "content/test/test_render_view_host.h"
 #include "ui/base/page_transition_types.h"
 
 struct FrameHostMsg_DidCommitProvisionalLoad_Params;
 
 namespace content {
@@ skipping 77 matching lines @@
   // Simulate a renderer-initiated navigation up until commit.
   void NavigateAndCommitRendererInitiated(int page_id,
                                           bool did_create_new_entry,
                                           const GURL& url);
 
   // With the current navigation logic this method is a no-op.
   // PlzNavigate: this method simulates receiving a BeginNavigation IPC.
   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();
 
   // If set, navigations will appear to have cleared the history list in the
   // RenderFrame
   // (FrameHostMsg_DidCommitProvisionalLoad_Params::history_list_was_cleared).
   // False by default.
   void set_simulate_history_list_was_cleared(bool cleared) {
     simulate_history_list_was_cleared_ = cleared;
@@ skipping 26 matching lines @@
                                   int response_code,
                                   const ModificationCallback& callback);
 
   // Computes the page ID for a pending navigation in this RenderFrameHost;
   int32_t ComputeNextPageID();
 
   TestRenderFrameHostCreationObserver child_creation_observer_;
 
   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_;
 
   DISALLOW_COPY_AND_ASSIGN(TestRenderFrameHost);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_TEST_TEST_RENDER_FRAME_HOST_H_