bluetoothLowEnergy: Send onServiceAdded after all characteristics are discovered

device/bluetooth/bluetooth_remote_gatt_characteristic_chromeos.cc

Index: device/bluetooth/bluetooth_remote_gatt_characteristic_chromeos.cc
diff --git a/device/bluetooth/bluetooth_remote_gatt_characteristic_chromeos.cc b/device/bluetooth/bluetooth_remote_gatt_characteristic_chromeos.cc
index 83231b9f6511bf060ee671249cc98ca6eb8af1cd..ad15a0698e8443c080368d05c5b5121bff9878e5 100644
--- a/device/bluetooth/bluetooth_remote_gatt_characteristic_chromeos.cc
+++ b/device/bluetooth/bluetooth_remote_gatt_characteristic_chromeos.cc
@@ -341,7 +341,7 @@ void BluetoothRemoteGattCharacteristicChromeOS::GattDescriptorAdded(
           GetProperties(object_path);
   DCHECK(properties);
   if (properties->characteristic.value() != object_path_) {
-    VLOG(2) << "Remote GATT descriptor does not belong to this characteristic.";
+    VLOG(3) << "Remote GATT descriptor does not belong to this characteristic.";
     return;
   }
 
@@ -356,7 +356,6 @@ void BluetoothRemoteGattCharacteristicChromeOS::GattDescriptorAdded(
   DCHECK(service_);
 
   service_->NotifyDescriptorAddedOrRemoved(this, descriptor, true /* added */);
-  service_->NotifyServiceChanged();
 }
 
 void BluetoothRemoteGattCharacteristicChromeOS::GattDescriptorRemoved(
@@ -374,12 +373,10 @@ void BluetoothRemoteGattCharacteristicChromeOS::GattDescriptorRemoved(
   DCHECK(descriptor->object_path() == object_path);
   descriptors_.erase(iter);
 
-  service_->NotifyDescriptorAddedOrRemoved(this, descriptor, false /* added */);
-  delete descriptor;
-
   DCHECK(service_);
+  service_->NotifyDescriptorAddedOrRemoved(this, descriptor, false /* added */);
 
-  service_->NotifyServiceChanged();
+  delete descriptor;
 }
 
 void BluetoothRemoteGattCharacteristicChromeOS::OnValueSuccess(