Implemented BluetoothTaskManagerWin class.

device/bluetooth/bluetooth_adapter_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_ADAPTER_WIN_H_
 #define DEVICE_BLUETOOTH_BLUETOOTH_ADAPTER_WIN_H_
 
 #include <string>
 
+#include "base/memory/scoped_ptr.h"
 #include "base/memory/weak_ptr.h"
 #include "device/bluetooth/bluetooth_adapter.h"
+#include "device/bluetooth/bluetooth_polling_thread_win.h"
+
+namespace base {
+
+class ThreadChecker;
+
+}  // namespace base
 
 namespace device {
 
 class BluetoothAdapterFactory;
 class BluetoothAdapterWinTest;
 class BluetoothDevice;
 
-class BluetoothAdapterWin : public BluetoothAdapter {
+class BluetoothAdapterWin : public BluetoothAdapter,
+                            private BluetoothPollingThreadWin::Observer {
  public:
   // BluetoothAdapter override
   virtual void AddObserver(BluetoothAdapter::Observer* observer) OVERRIDE;
   virtual void RemoveObserver(BluetoothAdapter::Observer* observer) OVERRIDE;
   virtual bool IsPresent() const OVERRIDE;
   virtual bool IsPowered() const OVERRIDE;
   virtual void SetPowered(
       bool powered,
       const base::Closure& callback,
       const ErrorCallback& error_callback) OVERRIDE;
   virtual bool IsDiscovering() const OVERRIDE;
   virtual void SetDiscovering(
       bool discovering,
       const base::Closure& callback,
       const ErrorCallback& error_callback) OVERRIDE;
   virtual ConstDeviceList GetDevices() const OVERRIDE;
   virtual BluetoothDevice* GetDevice(const std::string& address) OVERRIDE;
   virtual const BluetoothDevice* GetDevice(
       const std::string& address) const OVERRIDE;
   virtual void ReadLocalOutOfBandPairingData(
       const BluetoothOutOfBandPairingDataCallback& callback,
       const ErrorCallback& error_callback) OVERRIDE;
 
+  // BluetoothPollingThreadWin override

[bryeung, 2012/11/27 20:37:04]

you probably mean ::Observer here

Also, should this really be public?

[youngki, 2012/11/27 21:43:34]

Done.

+  virtual void AdapterStateChanged(
+      const BluetoothPollingThreadWin::AdapterState& state) OVERRIDE;
+
+  void SetBluetoothPollingThreadForTest(BluetoothPollingThreadWin* manager);
+
  protected:
   BluetoothAdapterWin();
   virtual ~BluetoothAdapterWin();
 
-  virtual void UpdateAdapterState();
-
  private:
   friend class BluetoothAdapterFactory;
   friend class BluetoothAdapterWinTest;
 
-  // Obtains the default adapter info (the first bluetooth radio info found on
-  // the system) and tracks future changes to it.
+  // Starts BluetoothPollingThreadWin thread to obtain the default adapter info.
   void TrackDefaultAdapter();
 
-  void PollAdapterState();
-
-  static const int kPollIntervalMs;
-
   bool powered_;
 
+  // Holding scoped_ptr guarantees that BluetoothAdapterWin will always outlive
+  // BluetoothPollingThreadWin thread.
+  scoped_ptr<BluetoothPollingThreadWin> thread_;
+
+  scoped_ptr<base::ThreadChecker> thread_checker_;

[bryeung, 2012/11/27 20:37:04]

why is this a pointer?

[youngki, 2012/11/27 21:43:34]

Because I want to forward-declare it instead of including header:
http://www.chromium.org/developers/coding-style/cpp-dos-and-donts#TOC-Forward....

[bryeung, 2012/11/28 13:41:11]

I think you are misinterpreting that section of the style guide.  The intention
is use forward-declaration when possible, not write the code specifically to
make forward declarations possible.  You should generally prefer value members
to pointers, unless the pointer is necessary for the implementation.

[youngki, 2012/11/28 15:04:39]

Done.

+
   // 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<BluetoothAdapterWin> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(BluetoothAdapterWin);
 };
 
 }  // namespace device
 
 #endif  // DEVICE_BLUETOOTH_BLUETOOTH_ADAPTER_WIN_H_