Add BluetoothGattCharacteristicTest::StartNotifySession_Reentrant unit test

device/bluetooth/bluetooth_low_energy_win_fake.cc

 // 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.
 
 #include "device/bluetooth/bluetooth_low_energy_win_fake.h"
 
 #include "base/strings/stringprintf.h"
 #include "device/bluetooth/bluetooth_low_energy_defs_win.h"
 
 namespace {
@@ skipping 238 matching lines @@
     PVOID context,
     BLUETOOTH_GATT_EVENT_HANDLE* out_handle) {
   // Right now, only CharacteristicValueChangedEvent is supported.
   CHECK(CharacteristicValueChangedEvent == type);
 
   scoped_ptr<GattCharacteristicObserver> observer(
       new GattCharacteristicObserver());
   observer->callback = callback;
   observer->context = context;
   *out_handle = (BLUETOOTH_GATT_EVENT_HANDLE)observer.get();
-  gatt_characteristic_observers_[*out_handle] = std::move(observer);
 
   PBLUETOOTH_GATT_VALUE_CHANGED_EVENT_REGISTRATION parameter =
       (PBLUETOOTH_GATT_VALUE_CHANGED_EVENT_REGISTRATION)event_parameter;
   for (USHORT i = 0; i < parameter->NumCharacteristics; i++) {
     GattCharacteristic* target_characteristic = GetSimulatedGattCharacteristic(
         service_path, &parameter->Characteristics[i]);
     CHECK(target_characteristic);
+
+    // Return error simulated by SimulateGattCharacteristicSetNotifyError.
+    if (target_characteristic->notify_errors.size()) {
+      HRESULT error = target_characteristic->notify_errors[0];
+      target_characteristic->notify_errors.erase(
+          target_characteristic->notify_errors.begin());
+      return error;
+    }
+
     target_characteristic->observers.push_back(*out_handle);
   }
+  gatt_characteristic_observers_[*out_handle] = std::move(observer);
 
   if (observer_)
     observer_->OnStartCharacteristicNotification();
 
   return S_OK;
 }
 
 HRESULT BluetoothLowEnergyWrapperFake::UnregisterGattEvent(
     BLUETOOTH_GATT_EVENT_HANDLE event_handle) {
   gatt_characteristic_observers_.erase(event_handle);
@@ skipping 154 matching lines @@
           characteristic->characteristic_info->AttributeHandle;
       event.CharacteristicValueDataSize =
           characteristic->value->DataSize + sizeof(ULONG);
       event.CharacteristicValue = characteristic->value.get();
       it->second->callback(CharacteristicValueChangedEvent, &event,
                            it->second->context);
     }
   }
 }
 
+void BluetoothLowEnergyWrapperFake::SimulateGattCharacteristicSetNotifyError(
+    GattCharacteristic* characteristic,
+    HRESULT error) {
+  characteristic->notify_errors.push_back(error);
+}
+
 void BluetoothLowEnergyWrapperFake::SimulateGattCharacteristicReadError(
     GattCharacteristic* characteristic,
     HRESULT error) {
   CHECK(characteristic);
   characteristic->read_errors.push_back(error);
 }
 
 void BluetoothLowEnergyWrapperFake::SimulateGattCharacteristicWriteError(
     GattCharacteristic* characteristic,
     HRESULT error) {
@@ skipping 118 matching lines @@
 std::string BluetoothLowEnergyWrapperFake::BluetoothAddressToCanonicalString(
     const BLUETOOTH_ADDRESS& btha) {
   std::string result = base::StringPrintf(
       "%02X:%02X:%02X:%02X:%02X:%02X", btha.rgBytes[5], btha.rgBytes[4],
       btha.rgBytes[3], btha.rgBytes[2], btha.rgBytes[1], btha.rgBytes[0]);
   return result;
 }
 
 }  // namespace win
 }  // namespace device