Simplified BluetoothAdapterWin.

device/bluetooth/bluetooth_task_manager_win_unittest.cc

Index: device/bluetooth/bluetooth_task_manager_win_unittest.cc
diff --git a/device/bluetooth/bluetooth_task_manager_win_unittest.cc b/device/bluetooth/bluetooth_task_manager_win_unittest.cc
index 4e461d04515dd6b2f54b9f12c4454cbade8b0f2c..8fd44d6e9160f45fec1820660b67443eaca2f52f 100644
--- a/device/bluetooth/bluetooth_task_manager_win_unittest.cc
+++ b/device/bluetooth/bluetooth_task_manager_win_unittest.cc
@@ -15,7 +15,9 @@ namespace {
 class BluetoothTaskObserver : public device::BluetoothTaskManagerWin::Observer {
  public:
   BluetoothTaskObserver() {
-    Clear();
+    num_adapter_state_changed_ = 0;
+    num_discovery_started_ = 0;
+    num_discovery_stopped_ = 0;

[keybuk, 2013/04/22 21:04:56]

Shouldn't these be initialized in the initializer list, rather than the body of
the constructor?

[youngki, 2013/04/23 02:57:54]

Done.

   }
 
   virtual ~BluetoothTaskObserver() {
@@ -34,19 +36,6 @@ class BluetoothTaskObserver : public device::BluetoothTaskManagerWin::Observer {
     num_discovery_stopped_++;
   }
 
-  virtual void DevicesDiscovered(
-      const ScopedVector<device::BluetoothTaskManagerWin::DeviceState>&
-          devices) OVERRIDE {
-    num_devices_discovered_++;
-  }
-
-  void Clear() {
-    num_adapter_state_changed_ = 0;
-    num_discovery_started_ = 0;
-    num_discovery_stopped_ = 0;
-    num_devices_discovered_ = 0;
-  }
-
   int num_adapter_state_changed() const {
     return num_adapter_state_changed_;
   }
@@ -59,15 +48,10 @@ class BluetoothTaskObserver : public device::BluetoothTaskManagerWin::Observer {
     return num_discovery_stopped_;
   }
 
-  int num_devices_discovered() const {
-    return num_devices_discovered_;
-  }
-
  private:
    int num_adapter_state_changed_;
    int num_discovery_started_;
    int num_discovery_stopped_;
-   int num_devices_discovered_;
 };
 
 }  // namespace