Bluetooth: mac: Crash while loging nil error.

device/bluetooth/bluetooth_low_energy_device_mac.mm

 // Copyright 2015 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_device_mac.h"
 
 #import <CoreFoundation/CoreFoundation.h>
 #include <stddef.h>
 
 #include "base/mac/mac_util.h"
@@ skipping 373 matching lines @@
        it != gatt_services_.end(); ++it) {
     device::BluetoothRemoteGattService* gatt_service = it->second;
     device::BluetoothRemoteGattServiceMac* gatt_service_mac =
         static_cast<BluetoothRemoteGattServiceMac*>(gatt_service);
     if (gatt_service_mac->GetService() == cb_service)
       return gatt_service_mac;
   }
   return nullptr;
 }
 
+void BluetoothLowEnergyDeviceMac::DidDisconnectPeripheral() {
+  RemoveAllServices();
+  if (create_gatt_connection_error_callbacks_.empty()) {
+    DidDisconnectGatt();
+  } else {
+    DidFailToConnectGatt(BluetoothDevice::ConnectErrorCode::ERROR_UNKNOWN);
+  }
+}
+
 void BluetoothLowEnergyDeviceMac::DidDisconnectPeripheral(
     BluetoothDevice::ConnectErrorCode error_code) {
-  SetGattServicesDiscoveryComplete(false);
-  // Removing all services at once to ensure that calling GetGattService on
-  // removed service in GattServiceRemoved returns null.
-  GattServiceMap gatt_services_swapped;
-  gatt_services_swapped.swap(gatt_services_);
-  gatt_services_swapped.clear();
+  RemoveAllServices();
   if (create_gatt_connection_error_callbacks_.empty()) {

[ortuno, 2016/08/09 15:40:25]

DidDisconnectPeripheral shouldn't be called after a connection attempt right?
Why are we checking the callbacks queue?

[jlebel, 2016/08/09 16:23:58]

I was guessing it disconnect is called before the connection is done,
DidDisconnectPeripheral could be called. But I tried it, it doesn't seem to
happen.

     // TODO(http://crbug.com/585897): Need to pass the error.
     DidDisconnectGatt();
   } else {
     DidFailToConnectGatt(error_code);
   }
 }
+
+void BluetoothLowEnergyDeviceMac::RemoveAllServices() {
+  SetGattServicesDiscoveryComplete(false);
+  // Removing all services at once to ensure that calling GetGattService on
+  // removed service in GattServiceRemoved returns null.
+  GattServiceMap gatt_services_swapped;
+  gatt_services_swapped.swap(gatt_services_);
+  gatt_services_swapped.clear();
+}