bluetooth: bluez: Fixed issue with missing notifications after reconnect.

device/bluetooth/bluez/bluetooth_device_bluez.cc

 // Copyright 2013 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/bluez/bluetooth_device_bluez.h"
 
 #include <stdio.h>
 
 #include <algorithm>
 #include <memory>
@@ skipping 910 matching lines @@
                                      weak_ptr_factory_.GetWeakPtr()));
 }
 
 void BluetoothDeviceBlueZ::OnSetTrusted(bool success) {
   LOG_IF(WARNING, !success) << object_path_.value()
                             << ": Failed to set device as trusted";
 }
 
 void BluetoothDeviceBlueZ::OnDisconnect(const base::Closure& callback) {
   VLOG(1) << object_path_.value() << ": Disconnected";
+  DidDisconnectGatt(false /* notifyDeviceChanged */);

[scheib, 2017/01/11 18:44:02]

Why false here?

[perja, 2017/01/12 09:26:28]

The bluez/dbus layer uses a property set (dbus::PropertySet) to keep track of
all properties for the connection. There is a separate OnPropertyChanged
callback fired when some of the properties change value. The notification for
changed device is handled through that mechanism. I will clarify with a comment
in the code.

   callback.Run();
 }
 
 void BluetoothDeviceBlueZ::OnDisconnectError(
     const ErrorCallback& error_callback,
     const std::string& error_name,
     const std::string& error_message) {
   LOG(WARNING) << object_path_.value()
                << ": Failed to disconnect device: " << error_name << ": "
                << error_message;
   error_callback.Run();
 }
 
 void BluetoothDeviceBlueZ::OnForgetError(const ErrorCallback& error_callback,
                                          const std::string& error_name,
                                          const std::string& error_message) {
   LOG(WARNING) << object_path_.value()
                << ": Failed to remove device: " << error_name << ": "
                << error_message;
   error_callback.Run();
 }
 
 }  // namespace bluez