MD Settings: Bluetooth: Fix adapter state and discovery

extensions/browser/api/bluetooth/bluetooth_api.cc

Index: extensions/browser/api/bluetooth/bluetooth_api.cc
diff --git a/extensions/browser/api/bluetooth/bluetooth_api.cc b/extensions/browser/api/bluetooth/bluetooth_api.cc
index 453ff77d62e6158dc56fd9de9feef7b3ee4220d0..7725bb20b8ce99e856dbbdc4cdc3201742878481 100644
--- a/extensions/browser/api/bluetooth/bluetooth_api.cc
+++ b/extensions/browser/api/bluetooth/bluetooth_api.cc
@@ -11,6 +11,7 @@
 #include "base/bind_helpers.h"
 #include "base/lazy_instance.h"
 #include "base/memory/ref_counted.h"
+#include "components/device_event_log/device_event_log.h"
 #include "content/public/browser/browser_thread.h"
 #include "device/bluetooth/bluetooth_adapter.h"
 #include "device/bluetooth/bluetooth_device.h"
@@ -64,6 +65,7 @@ BluetoothAPI* BluetoothAPI::Get(BrowserContext* context) {
 BluetoothAPI::BluetoothAPI(content::BrowserContext* context)
     : browser_context_(context) {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
+  BLUETOOTH_LOG(EVENT) << "BluetoothAPI: " << browser_context_;

[michaelpg, 2017/04/07 21:47:03]

is it normal to print a pointer?

[stevenjb, 2017/04/07 23:36:05]

I don't know about 'normal' but we do it. It can be helpful for identifying
which instance(s) are alive when there may be more than one.

   EventRouter* event_router = EventRouter::Get(browser_context_);
   event_router->RegisterObserver(this,
                                  bluetooth::OnAdapterStateChanged::kEventName);
@@ -72,11 +74,14 @@ BluetoothAPI::BluetoothAPI(content::BrowserContext* context)
   event_router->RegisterObserver(this, bluetooth::OnDeviceRemoved::kEventName);
 }
 
-BluetoothAPI::~BluetoothAPI() {}
+BluetoothAPI::~BluetoothAPI() {
+  BLUETOOTH_LOG(EVENT) << "~BluetoothAPI: " << browser_context_;
+}
 
 BluetoothEventRouter* BluetoothAPI::event_router() {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
   if (!event_router_) {
+    BLUETOOTH_LOG(EVENT) << "BluetoothAPI: Creating BluetoothEventRouter";
     event_router_.reset(new BluetoothEventRouter(browser_context_));
   }
   return event_router_.get();
@@ -84,6 +89,7 @@ BluetoothEventRouter* BluetoothAPI::event_router() {
 
 void BluetoothAPI::Shutdown() {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
+  BLUETOOTH_LOG(EVENT) << "BluetoothAPI: Shutdown";
   EventRouter::Get(browser_context_)->UnregisterObserver(this);
 }