Implement BluetoothRemoteGattServiceWin and related unit tests

device/bluetooth/bluetooth_low_energy_win_fake.h

 // Copyright 2016 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_FAKE_H_
 #define DEVICE_BLUETOOTH_BLUETOOTH_LOW_ENERGY_WIN_FAKE_H_
 
 #include "device/bluetooth/bluetooth_low_energy_win.h"
 
 #include <set>
@@ skipping 50 matching lines @@
   scoped_ptr<BTH_LE_GATT_DESCRIPTOR> descriptor_info;
   scoped_ptr<BTH_LE_GATT_DESCRIPTOR_VALUE> value;
 };
 
 // Fake implementation of BluetoothLowEnergyWrapper. Used for BluetoothTestWin.
 class BluetoothLowEnergyWrapperFake : public BluetoothLowEnergyWrapper {
  public:
   BluetoothLowEnergyWrapperFake();
   ~BluetoothLowEnergyWrapperFake() override;
 
+  bool IsBluetoothLowEnergySupported() override;
   bool EnumerateKnownBluetoothLowEnergyDevices(
       ScopedVector<BluetoothLowEnergyDeviceInfo>* devices,
       std::string* error) override;
   bool EnumerateKnownBluetoothLowEnergyGattServiceDevices(
       ScopedVector<BluetoothLowEnergyDeviceInfo>* devices,
       std::string* error) override;
   bool EnumerateKnownBluetoothLowEnergyServices(
       const base::FilePath& device_path,
       ScopedVector<BluetoothLowEnergyServiceInfo>* services,
       std::string* error) override;
+  HRESULT ReadIncludedServicesOfAService(
+      base::FilePath& service_path,
+      const PBTH_LE_GATT_SERVICE service,
+      scoped_ptr<BTH_LE_GATT_SERVICE>* out_included_services,
+      USHORT* out_counts) override;
+  HRESULT ReadCharacteristicsOfAService(
+      base::FilePath& service_path,
+      const PBTH_LE_GATT_SERVICE service,
+      scoped_ptr<BTH_LE_GATT_CHARACTERISTIC>* out_included_characteristics,
+      USHORT* out_counts) override;
 
   BLEDevice* SimulateBLEDevice(std::string device_name,
                                BLUETOOTH_ADDRESS device_address);
   BLEDevice* GetSimulatedBLEDevice(std::string device_address);
-  BLEGattService* SimulateBLEGattService(BLEDevice* device, std::string uuid);
+
+  // Note: |parent_service| == nullptr if simulated service is a primary

[scheib, 2016/02/24 01:39:10]

|parent_service| may be nullptr to indicate a primary service.

[gogerald1, 2016/02/24 16:09:06]

Done.

+  // service.
+  BLEGattService* SimulateBLEGattService(BLEDevice* device,
+                                         BLEGattService* parent_service,
+                                         const BTH_LE_UUID& uuid);
+
+  // Note: |parent_service| == nullptr if removed service is a primary

[scheib, 2016/02/24 01:39:10]

ditto

[gogerald1, 2016/02/24 16:09:06]

Done.

+  // service.
+  void SimulateBLEGattServiceRemove(BLEDevice* device,

[scheib, 2016/02/24 01:39:10]

SimulateBLEGattServiceRemoved

[gogerald1, 2016/02/24 16:09:06]

Done.

+                                    BLEGattService* parent_service,
+                                    std::string attribute_handle);
+
+  // Note: |chain_of_att_handle| contains the attribute handles of the services
+  // in order from primary service to target service. The last item in
+  // |chain_of_att_handle| is the target service's attribute handle.
+  BLEGattService* GetSimulatedGattService(
+      BLEDevice* device,
+      const std::vector<std::string>& chain_of_att_handle);
+  BLEGattCharacteristic* SimulateCharacterisc(

[scheib, 2016/02/24 01:39:10]

SimulateCharacteristic. Note this doesn't follow the above names with 'BLEGatt'
in them. I think the 'BLEGatt' is redundant, you may want to refactor that out
in a later patch.

[gogerald1, 2016/02/24 16:09:06]

Follow above names "BLEGatt" for now since all the other data structures and
interfaces followed. Refactor them out in a separate path at the end. 

+      std::string device_address,
+      BLEGattService* parent_service,
+      const BTH_LE_GATT_CHARACTERISTIC& characteristic);
+  void SimulateCharacteriscRemove(BLEGattService* parent_service,
+                                  std::string attribute_handle);
 
  private:
   // Generate an unique attribute handle on |device_address|.
   USHORT GenerateAUniqueAttributeHandle(std::string device_address);
 
   // Generate device path for the BLE device with |device_address|.
   base::string16 GenerateBLEDevicePath(std::string device_address);
 
   // Generate GATT service device path of the service with
   // |service_attribute_handle|. |resident_device_path| is the BLE device this
   // GATT service belongs to.
   base::string16 GenerateBLEGattServiceDevicePath(
       base::string16 resident_device_path,
       USHORT service_attribute_handle);
 
   // Extract device address from the device |path| generated by
   // GenerateBLEDevicePath or GenerateBLEGattServiceDevicePath.
   base::string16 ExtractDeviceAddressFromDevicePath(base::string16 path);
 
-  // Extract service attribute handle from the |path| generated by
+  // Extract service attribute handles from the |path| generated by
   // GenerateBLEGattServiceDevicePath.
-  base::string16 ExtractServiceAttributeHandleFromDevicePath(
+  std::vector<std::string> ExtractServiceAttributeHandlesFromDevicePath(
       base::string16 path);
 
   // The canonical BLE device address string format is the
   // BluetoothDevice::CanonicalizeAddress.
   std::string BluetoothAddressToCanonicalString(const BLUETOOTH_ADDRESS& btha);
 
-  // The canonical UUID string format is device::BluetoothUUID.value().
-  BTH_LE_UUID CanonicalStringToBTH_LE_UUID(std::string uuid);
-
   // Table to store allocated attribute handle for a device.
   BLEAttributeHandleTable attribute_handle_table_;
   BLEDevicesMap simulated_devices_;
 };
 
 }  // namespace win
 }  // namespace device
 
 #endif  // DEVICE_BLUETOOTH_BLUETOOTH_LOW_ENERGY_WIN_FAKE_H_