diff options
| author | armansito@chromium.org <armansito@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-06-04 22:28:05 +0000 |
|---|---|---|
| committer | armansito@chromium.org <armansito@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-06-04 22:28:05 +0000 |
| commit | 7189f7f71e868d910da86b594e4e66e603547f06 (patch) | |
| tree | d7839ef50b30db36abfe2da76e9f9862c7ad470c /device | |
| parent | 0e60fce0f20f6b7cfadf52fb80cef888f007a2fe (diff) | |
| download | chromium_src-7189f7f71e868d910da86b594e4e66e603547f06.zip chromium_src-7189f7f71e868d910da86b594e4e66e603547f06.tar.gz chromium_src-7189f7f71e868d910da86b594e4e66e603547f06.tar.bz2 | |
device/bluetooth: Fix bug in the GattServiceChanged event is sent.
This CL fixes a bug in which a BluetoothRemoteGattServiceChromeOS sent a
BluetoothGattService::Observer::GattServiceChanged event even when the
underlying D-Bus object that triggered the property change signal did not
correspond to the instance in question. This is fixed by correctly checking
that the object paths match.
BUG=none
R=keybuk@chromium.org
Review URL: https://codereview.chromium.org/313013003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@274936 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'device')
| -rw-r--r-- | device/bluetooth/bluetooth_remote_gatt_service_chromeos.cc | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/device/bluetooth/bluetooth_remote_gatt_service_chromeos.cc b/device/bluetooth/bluetooth_remote_gatt_service_chromeos.cc index f2ec861..e4b0ae2 100644 --- a/device/bluetooth/bluetooth_remote_gatt_service_chromeos.cc +++ b/device/bluetooth/bluetooth_remote_gatt_service_chromeos.cc @@ -194,6 +194,10 @@ void BluetoothRemoteGattServiceChromeOS::NotifyDescriptorValueChanged( void BluetoothRemoteGattServiceChromeOS::GattServicePropertyChanged( const dbus::ObjectPath& object_path, const std::string& property_name){ + if (object_path != object_path_) + return; + + VLOG(1) << "Service property changed: " << object_path.value(); NotifyServiceChanged(); } |