Implemented BluetoothTaskManagerWin class.

device/bluetooth/bluetooth_polling_thread_win.h

+// 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.
+
+#ifndef DEVICE_BLUETOOTH_BLUETOOTH_POLLING_THREAD_WIN_H_
+#define DEVICE_BLUETOOTH_BLUETOOTH_POLLING_THREAD_WIN_H_
+
+#include "base/memory/scoped_ptr.h"
+#include "base/synchronization/cancellation_flag.h"
+#include "base/threading/thread.h"
+#include "base/threading/thread_checker.h"
+
+namespace base {
+
+class CancellationFlag;
+class ThreadChecker;

[bryeung, 2012/11/28 16:26:08]

not needed with the headers above

[youngki, 2012/11/28 19:19:13]

Done.

+
+}  // namespace base
+
+namespace device {
+
+// A thread dedicated to perform blocking bluetooth APIs on Windows.
+class BluetoothPollingThreadWin : public base::Thread {
+ public:
+  struct AdapterState {
+    std::string name;
+    std::string address;
+    bool powered;
+  };
+
+  class Observer {
+   public:
+    virtual ~Observer() {}
+
+    // This will always be called on the UI thread.
+    virtual void AdapterStateChanged(const AdapterState& state) {}
+  };
+
+  BluetoothPollingThreadWin();
+  virtual ~BluetoothPollingThreadWin();
+
+  void AddObserver(Observer* observer);
+  void RemoveObserver(Observer* observer);
+
+  void Init();
+
+ private:
+  // Populates bluetooth adapter state using adapter_handle.
+  static void GetAdapterState(const HANDLE adapter_handle,

[bryeung, 2012/11/28 16:26:08]

this should go inside an anonymous namespace in the .cc file if there is no
reason for it be part of the this class

[youngki, 2012/11/28 19:19:13]

Done.

+                              AdapterState* state);
+
+  // Polls Windows Bluetooth API on BluetoothPollingThread.

[bryeung, 2012/11/28 16:26:08]

More descriptive comments would be nice, e.g.:
"Called periodically to get the latest state of the Bluetooth Adapter from the
Windows APIs.  Posts another call to PollAdapter unless |cancellation_flag_| is
set."

[youngki, 2012/11/28 19:19:13]

Done.

+  void PollAdapter(base::WeakPtr<BluetoothPollingThreadWin> weak_ptr);
+
+  // Callback to be run on the UI thread.

[bryeung, 2012/11/28 16:26:08]

"Notify all Observers of updated AdapterState.  Should only be called on the UI
thread."

[youngki, 2012/11/28 19:19:13]

Done.

+  void OnAdapterStateChanged(const AdapterState* state);
+
+  static const int kPollIntervalMs;
+
+  // Polling will be canceled when BluetoothPollingThread terminates.
+  base::CancellationFlag cancellation_flag_;
+
+  // Checks that PollAdapter runs on BluetoothPollingThread.
+  base::ThreadChecker thread_checker_;
+
+  // List of observers interested in event notifications from us.

[bryeung, 2012/11/28 16:26:08]

s/from us//

[youngki, 2012/11/28 19:19:13]

Done.

+  ObserverList<Observer> observers_;
+
+  // NOTE: This should remain the last member so it'll be destroyed and
+  // invalidate its weak pointers before any other members are destroyed.
+  base::WeakPtrFactory<BluetoothPollingThreadWin> weak_ptr_factory_;
+
+  DISALLOW_COPY_AND_ASSIGN(BluetoothPollingThreadWin);
+};
+
+}  // namespace device
+
+#endif  // DEVICE_BLUETOOTH_BLUETOOTH_POLLING_THREAD_WIN_H_