bluetoothLowEnergy: Send onServiceAdded after all characteristics are discovered

device/bluetooth/bluetooth_remote_gatt_characteristic_chromeos.cc

 // Copyright 2014 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_remote_gatt_characteristic_chromeos.h"
 
 #include <limits>
 
 #include "base/logging.h"
 #include "base/strings/stringprintf.h"
@@ skipping 323 matching lines @@
     VLOG(1) << "Remote GATT characteristic descriptor already exists: "
             << object_path.value();
     return;
   }
 
   BluetoothGattDescriptorClient::Properties* properties =
       DBusThreadManager::Get()->GetBluetoothGattDescriptorClient()->
           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;
   }
 
   VLOG(1) << "Adding new remote GATT descriptor for GATT characteristic: "
           << GetIdentifier() << ", UUID: " << GetUUID().canonical_value();
 
   BluetoothRemoteGattDescriptorChromeOS* descriptor =
       new BluetoothRemoteGattDescriptorChromeOS(this, object_path);
   descriptors_[object_path] = descriptor;
   DCHECK(descriptor->GetIdentifier() == object_path.value());
   DCHECK(descriptor->GetUUID().IsValid());
   DCHECK(service_);
 
   service_->NotifyDescriptorAddedOrRemoved(this, descriptor, true /* added */);
-  service_->NotifyServiceChanged();
 }
 
 void BluetoothRemoteGattCharacteristicChromeOS::GattDescriptorRemoved(
     const dbus::ObjectPath& object_path) {
   DescriptorMap::iterator iter = descriptors_.find(object_path);
   if (iter == descriptors_.end()) {
     VLOG(2) << "Unknown descriptor removed: " << object_path.value();
     return;
   }
 
   VLOG(1) << "Removing remote GATT descriptor from characteristic: "
           << GetIdentifier() << ", UUID: " << GetUUID().canonical_value();
 
   BluetoothRemoteGattDescriptorChromeOS* descriptor = iter->second;
   DCHECK(descriptor->object_path() == object_path);
   descriptors_.erase(iter);
 
+  DCHECK(service_);
   service_->NotifyDescriptorAddedOrRemoved(this, descriptor, false /* added */);
+
   delete descriptor;
-
-  DCHECK(service_);
-
-  service_->NotifyServiceChanged();
 }
 
 void BluetoothRemoteGattCharacteristicChromeOS::OnValueSuccess(
     const ValueCallback& callback,
     const std::vector<uint8>& value) {
   VLOG(1) << "Characteristic value read: " << value;
   cached_value_ = value;
 
   DCHECK(service_);
   service_->NotifyCharacteristicValueChanged(this, cached_value_);
@@ skipping 77 matching lines @@
 
 void BluetoothRemoteGattCharacteristicChromeOS::ProcessStartNotifyQueue() {
   while (!pending_start_notify_calls_.empty()) {
     PendingStartNotifyCall callbacks = pending_start_notify_calls_.front();
     pending_start_notify_calls_.pop();
     StartNotifySession(callbacks.first, callbacks.second);
   }
 }
 
 }  // namespace chromeos