Implemented BluetoothAdapterWin::IsPresent().

device/bluetooth/bluetooth_adapter_win.cc

 // Copyright (c) 2012 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.
 //
 // TODO(youngki): Implement this file.
 
 #include "device/bluetooth/bluetooth_adapter_win.h"
 
+#include <BluetoothAPIs.h>
 #include <string>
+#include "base/bind.h"
 #include "base/logging.h"
+#include "base/message_loop.h"
+#include "base/stringprintf.h"
+#include "base/sys_string_conversions.h"
+
+# pragma comment(lib, "Bthprops.lib")
+
+namespace {
+
+const BLUETOOTH_FIND_RADIO_PARAMS bluetooth_adapter_param =
+    { sizeof(BLUETOOTH_FIND_RADIO_PARAMS) };
+
+}  // namespace
 
 namespace device {
 
+const int BluetoothAdapterWin::kPollIntervalMs = 500;
+
 BluetoothAdapterWin::BluetoothAdapterWin()
     : BluetoothAdapter(),
       ALLOW_THIS_IN_INITIALIZER_LIST(weak_ptr_factory_(this)) {
 }
 
 BluetoothAdapterWin::~BluetoothAdapterWin() {
 }
 
 void BluetoothAdapterWin::AddObserver(BluetoothAdapter::Observer* observer) {
   NOTIMPLEMENTED();
 }
 
 void BluetoothAdapterWin::RemoveObserver(BluetoothAdapter::Observer* observer) {
   NOTIMPLEMENTED();
 }
 
 bool BluetoothAdapterWin::IsPresent() const {
-  NOTIMPLEMENTED();
-  return false;
+  return !address_.empty();
 }
 
 bool BluetoothAdapterWin::IsPowered() const {
   NOTIMPLEMENTED();
   return false;
 }
 
 void BluetoothAdapterWin::SetPowered(
     bool powered,
     const base::Closure& callback,
@@ skipping 28 matching lines @@
   NOTIMPLEMENTED();
   return NULL;
 }
 
 void BluetoothAdapterWin::ReadLocalOutOfBandPairingData(
     const BluetoothOutOfBandPairingDataCallback& callback,
     const ErrorCallback& error_callback) {
   NOTIMPLEMENTED();
 }
 
+void BluetoothAdapterWin::TrackDefaultAdapter() {
+  PollAdapterState();
+}
+
+void BluetoothAdapterWin::UpdateAdapterState() {
+  HBLUETOOTH_RADIO_FIND bluetooth_adapter_handle = NULL;
+  BLUETOOTH_RADIO_INFO bluetooth_adapter_info =
+      { sizeof(BLUETOOTH_RADIO_INFO), 0 };
+  BluetoothFindFirstRadio(&bluetooth_adapter_param, &bluetooth_adapter_handle);
+  if (bluetooth_adapter_handle &&
+      ERROR_SUCCESS == BluetoothGetRadioInfo(bluetooth_adapter_handle,
+                                             &bluetooth_adapter_info)) {
+    name_ = base::SysWideToUTF8(bluetooth_adapter_info.szName);
+    address_ = base::StringPrintf("%02X:%02X:%02X:%02X:%02X:%02X",
+        bluetooth_adapter_info.address.rgBytes[5],
+        bluetooth_adapter_info.address.rgBytes[4],
+        bluetooth_adapter_info.address.rgBytes[3],
+        bluetooth_adapter_info.address.rgBytes[2],
+        bluetooth_adapter_info.address.rgBytes[1],
+        bluetooth_adapter_info.address.rgBytes[0]);
+  } else {
+    name_.clear();
+    address_.clear();
+  }
+}
+
+void BluetoothAdapterWin::PollAdapterState() {
+  UpdateAdapterState();
+
+  MessageLoop::current()->PostDelayedTask(
+      FROM_HERE,
+      base::Bind(&BluetoothAdapterWin::PollAdapterState,
+                 weak_ptr_factory_.GetWeakPtr()),

[bryeung, 2012/11/09 21:55:48]

Since this is a weak_ptr, couldn't the adapter have already been destroyed by
the time this method is called, leading to a crash due to a NULL this?

[youngki, 2012/11/12 16:42:53]

I ran a simple test by queuing this to message loop and destroying the adapter,
and the program handled it gracefully by simply not running it.

Also I asked to chromium-dev and I think in fact base::Bind will only run a task
if the given weak_ptr is non-null:
https://groups.google.com/a/chromium.org/d/topic/chromium-dev/l1PfCT_3fvI/dis...

+      base::TimeDelta::FromMilliseconds(kPollIntervalMs));
+}
+
 }  // namespace device