bluetooth: Remove indicator only when there are no more devices connected

content/browser/web_contents/web_contents_impl.h

Index: content/browser/web_contents/web_contents_impl.h
diff --git a/content/browser/web_contents/web_contents_impl.h b/content/browser/web_contents/web_contents_impl.h
index 384bdd307a89a215f2b7c2fa10f1f6939e7a1126..b7171319a30ffc7f73dfd4a1dacb53d38f373417 100644
--- a/content/browser/web_contents/web_contents_impl.h
+++ b/content/browser/web_contents/web_contents_impl.h
@@ -277,8 +277,9 @@ class CONTENT_EXPORT WebContentsImpl
   int GetCapturerCount() const override;
   bool IsAudioMuted() const override;
   void SetAudioMuted(bool mute) override;
-  bool IsBluetoothDeviceConnected() const override;
-  void SetBluetoothDeviceConnected(bool connected) override;
+  void IncrementBluetoothConnectedDeviceCount() override;
+  void DecrementBluetoothConnectedDeviceCount() override;

[ncarter (slow), 2016/04/07 19:48:54]

Looks like the increment/decrement methods are only called from inside content;
let's remove them from the public interface, and make them non-virtual methods
of WebContentsImpl.

(bluetooth_dispatcher_host, being inside of content, can freely static_cast its
web_contents to a WebContentsImpl).

[ortuno, 2016/04/07 22:34:51]

Done.

+  bool IsConnectedToBluetoothDevice() const override;
   bool IsCrashed() const override;
   void SetIsCrashed(base::TerminationStatus status, int error_code) override;
   base::TerminationStatus GetCrashedStatus() const override;
@@ -1308,7 +1309,7 @@ class CONTENT_EXPORT WebContentsImpl
   // Created on-demand to mute all audio output from this WebContents.
   scoped_ptr<WebContentsAudioMuter> audio_muter_;
 
-  bool bluetooth_device_connected_;
+  size_t bluetooth_connected_device_count_;
 
   bool virtual_keyboard_requested_;