Add BluetoothGattCharacteristicTest::StartNotifySession_Reentrant unit test

device/bluetooth/test/bluetooth_test_win.cc

Index: device/bluetooth/test/bluetooth_test_win.cc
diff --git a/device/bluetooth/test/bluetooth_test_win.cc b/device/bluetooth/test/bluetooth_test_win.cc
index 23ea410f261a37e1b2b1824c01b37ab0c9dce5a8..02536bb9df9f8440f6d39c42becb72bea93f7f82 100644
--- a/device/bluetooth/test/bluetooth_test_win.cc
+++ b/device/bluetooth/test/bluetooth_test_win.cc
@@ -381,6 +381,17 @@ void BluetoothTestWin::SimulateGattNotifySessionStarted(
   FinishPendingTasks();
 }
 
+void BluetoothTestWin::SimulateGattNotifySessionStartError(
+    BluetoothRemoteGattCharacteristic* characteristic,
+    BluetoothRemoteGattService::GattErrorCode error_code) {
+  win::GattCharacteristic* simulated_characteristic =
+      GetSimulatedCharacteristic(characteristic);
+  CHECK(simulated_characteristic);

[ortuno, 2016/04/28 16:21:25]

DCHECK is enough. CHECK is normally used if failing the condition would result
in a security vulnerability:

https://www.chromium.org/developers/coding-style#TOC-CHECK-DCHECK-and-NOTREAC...

[gogerald1, 2016/05/02 15:37:50]

Done.

+  CHECK(error_code == BluetoothRemoteGattService::GATT_ERROR_UNKNOWN);
+  fake_bt_le_wrapper_->SimulateGattCharacteristicSetNotifyError(
+      simulated_characteristic, E_BLUETOOTH_ATT_UNKNOWN_ERROR);
+}
+
 void BluetoothTestWin::SimulateGattCharacteristicChanged(
     BluetoothRemoteGattCharacteristic* characteristic,
     const std::vector<uint8_t>& value) {