Invoke GattDiscoveryCompleteForService by observing ServicesResolved property

device/bluetooth/bluez/bluetooth_device_bluez.h

Index: device/bluetooth/bluez/bluetooth_device_bluez.h
diff --git a/device/bluetooth/bluez/bluetooth_device_bluez.h b/device/bluetooth/bluez/bluetooth_device_bluez.h
index 293c2466b6dafe10bbf817fd9160660f3b179563..1cbe3c35451fc4f226d09474e9b2c83fc61cd061 100644
--- a/device/bluetooth/bluez/bluetooth_device_bluez.h
+++ b/device/bluetooth/bluez/bluetooth_device_bluez.h
@@ -38,6 +38,7 @@ class BluetoothPairingBlueZ;
 // thread.
 class DEVICE_BLUETOOTH_EXPORT BluetoothDeviceBlueZ
     : public device::BluetoothDevice,
+      public bluez::BluetoothDeviceClient::Observer,
       public bluez::BluetoothGattServiceClient::Observer {
  public:
   // BluetoothDevice override
@@ -124,10 +125,18 @@ class DEVICE_BLUETOOTH_EXPORT BluetoothDeviceBlueZ
       scoped_refptr<device::BluetoothSocketThread> socket_thread);
   ~BluetoothDeviceBlueZ() override;
 
-  // bluez::BluetoothGattServiceClient::Observer overrides.
+  // bluez::BluetoothDeviceClient::Observer overrides
+  void DevicePropertyChanged(const dbus::ObjectPath& object_path,
+                             const std::string& property_name) override;
+
+  // bluez::BluetoothGattServiceClient::Observer overrides
   void GattServiceAdded(const dbus::ObjectPath& object_path) override;
   void GattServiceRemoved(const dbus::ObjectPath& object_path) override;
 
+  // Called conditionally by the constructor and |DevicePropertyChanged()| to

[scheib, 2016/05/16 20:22:26]

Currently this is only called by constructor. See comments at the definition and
then adjust the comment to reflect what you decide to go with.

[Miao, 2016/05/17 18:08:27]

Done.

+  // initialize the map of GATT services associated with this device.
+  void InitializeGattServiceMap();
+
   // Called by dbus:: on completion of the D-Bus method call to get the
   // connection attributes of the current connection to the device.
   void OnGetConnInfo(const ConnectionInfoCallback& callback,
@@ -206,6 +215,11 @@ class DEVICE_BLUETOOTH_EXPORT BluetoothDeviceBlueZ
   scoped_refptr<base::SequencedTaskRunner> ui_task_runner_;
   scoped_refptr<device::BluetoothSocketThread> socket_thread_;
 
+  // This vector temporarily cached the newly added services until

[rkc, 2016/05/16 16:25:23]

nit: s/cached/caches

[Miao, 2016/05/17 00:22:47]

Done.

+  // ServicesResolved property becomes true.
+  std::vector<device::BluetoothRemoteGattService*>
+      newly_discovered_gatt_services_;
+
   // During pairing this is set to an object that we don't own, but on which
   // we can make method calls to request, display or confirm PIN Codes and
   // Passkeys. Generally it is the object that owns this one.