bluetoothLowEnergy: Send onServiceAdded after all characteristics are discovered

device/bluetooth/bluetooth_gatt_service.h

 // 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.
 
 #ifndef DEVICE_BLUETOOTH_BLUETOOTH_GATT_SERVICE_H_
 #define DEVICE_BLUETOOTH_BLUETOOTH_GATT_SERVICE_H_
 
 #include <vector>
 
 #include "base/basictypes.h"
@@ skipping 113 matching lines @@
         const ValueCallback& callback,
         const ErrorCallback& error_callback) = 0;
   };
 
   // Interface for observing changes from a BluetoothGattService. Properties
   // of remote services are received asynchronously. The Observer interface can
   // be used to be notified when the initial values of a service are received
   // as well as when successive changes occur during its life cycle.
   class Observer {
    public:
+    // Called when all characteristic and descriptor discovery procedures are
+    // known to be completed for the GATT service |service|. This method will be
+    // called after the initial discovery of a GATT service and will usually be
+    // preceded by calls to GattCharacteristicAdded and GattDescriptorAdded.
+    virtual void GattDiscoveryCompleteForService(
+        BluetoothGattService* service) {}
+
     // Called when properties of the remote GATT service |service| have changed.
     // This will get called for properties such as UUID, as well as for changes
     // to the list of known characteristics and included services. Observers
     // should read all GATT characteristic and descriptors objects and do any
-    // necessary set up required for a changed service. This method may be
-    // called several times, especially when the service is discovered for the
-    // first time. It will be called for each characteristic and characteristic
-    // descriptor that is discovered or removed. Hence this method should be
-    // used to check whether or not all characteristics of a service have been
-    // discovered that correspond to the profile implemented by the Observer.
+    // necessary set up required for a changed service.
     virtual void GattServiceChanged(BluetoothGattService* service) {}
 
     // Called when the remote GATT characteristic |characteristic| belonging to
     // GATT service |service| has been discovered. Use this to issue any initial
     // read/write requests to the characteristic but don't cache the pointer as
     // it may become invalid. Instead, use the specially assigned identifier
     // to obtain a characteristic and cache that identifier as necessary, as it
     // can be used to retrieve the characteristic from its GATT service. The
     // number of characteristics with the same UUID belonging to a service
     // depends on the particular profile the remote device implements, hence the
     // client of a GATT based profile will usually operate on the whole set of
     // characteristics and not just one.
-    //
-    // This method will always be followed by a call to GattServiceChanged,
-    // which can be used by observers to get all the characteristics of a
-    // service and perform the necessary updates. GattCharacteristicAdded exists
-    // mostly for convenience.
     virtual void GattCharacteristicAdded(
         BluetoothGattService* service,
         BluetoothGattCharacteristic* characteristic) {}
 
     // Called when a GATT characteristic |characteristic| belonging to GATT
-    // service |service| has been removed. This method is for convenience
-    // and will be followed by a call to GattServiceChanged (except when called
-    // after the service gets removed) which should be used for bootstrapping a
-    // GATT based profile. See the documentation of GattCharacteristicAdded and
-    // GattServiceChanged for more information. Try to obtain the service from
-    // its device to see whether or not the service has been removed.
+    // service |service| has been removed.
     virtual void GattCharacteristicRemoved(
         BluetoothGattService* service,
         BluetoothGattCharacteristic* characteristic) {}
 
     // Called when the remote GATT characteristic descriptor |descriptor|
     // belonging to characteristic |characteristic| has been discovered. Don't
     // cache the arguments as the pointers may become invalid. Instead, use the
     // specially assigned identifier to obtain a descriptor and cache that
     // identifier as necessary.
-    //
-    // This method will always be followed by a call to GattServiceChanged,
-    // which can be used by observers to get all the characteristics of a
-    // service and perform the necessary updates. GattDescriptorAdded exists
-    // mostly for convenience.
     virtual void GattDescriptorAdded(
         BluetoothGattCharacteristic* characteristic,
         BluetoothGattDescriptor* descriptor) {}
 
     // Called when a GATT characteristic descriptor |descriptor| belonging to
-    // characteristic |characteristic| has been removed. This method is for
-    // convenience and will be followed by a call to GattServiceChanged (except
-    // when called after the service gets removed).
+    // characteristic |characteristic| has been removed.
     virtual void GattDescriptorRemoved(
         BluetoothGattCharacteristic* characteristic,
         BluetoothGattDescriptor* descriptor) {}
 
     // Called when the value of a characteristic has changed. This might be a
     // result of a read/write request to, or a notification/indication from, a
     // remote GATT characteristic.
     virtual void GattCharacteristicValueChanged(
         BluetoothGattService* service,
         BluetoothGattCharacteristic* characteristic,
@@ skipping 94 matching lines @@
  protected:
   BluetoothGattService();
 
  private:
   DISALLOW_COPY_AND_ASSIGN(BluetoothGattService);
 };
 
 }  // namespace device
 
 #endif  // DEVICE_BLUETOOTH_BLUETOOTH_GATT_SERVICE_H_