Refactor bluetooth_low_energy_win to prepare for new Bluetooth test fixture

device/bluetooth/bluetooth_low_energy_win.h

 // Copyright 2014 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_LOW_ENERGY_WIN_H_
 #define DEVICE_BLUETOOTH_BLUETOOTH_LOW_ENERGY_WIN_H_
 
 #include <stddef.h>
 #include <stdint.h>
 
@@ skipping 80 matching lines @@
 
   base::FilePath path;
   std::string id;
   std::string friendly_name;
   BLUETOOTH_ADDRESS address;
   bool visible;
   bool authenticated;
   bool connected;
 };
 
-// Enumerates the list of known (i.e. already paired) Bluetooth LE devices on
-// this machine. In case of error, returns false and sets |error| with an error
-// message describing the problem.
-// Note: This function returns an error if Bluetooth Low Energy is not supported
-// on this Windows platform.
-bool EnumerateKnownBluetoothLowEnergyDevices(
-    ScopedVector<BluetoothLowEnergyDeviceInfo>* devices,
-    std::string* error);
-
-// Enumerates the list of known (i.e. cached) GATT services for a given
-// Bluetooth LE device |device_path| into |services|. In case of error, returns
-// false and sets |error| with an error message describing the problem. Note:
-// This function returns an error if Bluetooth Low Energy is not supported on
-// this Windows platform.
-bool EnumerateKnownBluetoothLowEnergyServices(
-    const base::FilePath& device_path,
-    ScopedVector<BluetoothLowEnergyServiceInfo>* services,
-    std::string* error);
-
 bool DEVICE_BLUETOOTH_EXPORT
 ExtractBluetoothAddressFromDeviceInstanceIdForTesting(
     const std::string& instance_id,
     BLUETOOTH_ADDRESS* btha,
     std::string* error);
 
+// Wrap Windows Apis to enumerate Bluetooth low energy devices and services.

[scheib, 2016/01/29 04:35:50]

Wraps Windows APIs used to access Bluetooth Low Energy devices, providing an
interface that can be replaced with fakes in tests.

[gogerald1, 2016/02/02 22:00:21]

Done.

+class BluetoothLowEnergyHub {

[scheib, 2016/01/29 04:35:50]

I think these names would be more descriptive:
BluetoothLowEnergyWrapper
BluetoothLowEnergyFake

[gogerald1, 2016/02/02 22:00:21]

Done.

+ public:
+  static BluetoothLowEnergyHub* GetInstance();
+  static void SetInstanceForTest(BluetoothLowEnergyHub* instance);
+
+  // Enumerates the list of known (i.e. already paired) Bluetooth LE devices on
+  // this machine. In case of error, returns false and sets |error| with an
+  // error message describing the problem.
+  // Note: This function returns an error if Bluetooth Low Energy is not
+  // supported on this Windows platform.
+  virtual bool EnumerateKnownBluetoothLowEnergyDevices(
+      ScopedVector<BluetoothLowEnergyDeviceInfo>* devices,
+      std::string* error);
+
+  // Enumerates the list of known (i.e. cached) GATT services for a given
+  // Bluetooth LE device |device_path| into |services|. In case of error,
+  // returns false and sets |error| with an error message describing the
+  // problem.
+  // Note: This function returns an error if Bluetooth Low Energy is not
+  // supported on this Windows platform.
+  virtual bool EnumerateKnownBluetoothLowEnergyServices(
+      const base::FilePath& device_path,
+      ScopedVector<BluetoothLowEnergyServiceInfo>* services,
+      std::string* error);
+
+ protected:
+  BluetoothLowEnergyHub();
+  virtual ~BluetoothLowEnergyHub();
+};
+
 }  // namespace win
 }  // namespace device
 
 #endif  // DEVICE_BLUETOOTH_BLUETOOTH_LOW_ENERGY_WIN_H_