diff options
| author | armansito <armansito@chromium.org> | 2014-09-05 10:49:34 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2014-09-05 17:52:01 +0000 |
| commit | ebff093d22cd5c0613f3493acdbc1af1cfd5d31f (patch) | |
| tree | 39994650df189224daec3bd24aa469cf17227abd /dbus/dbus.gyp | |
| parent | 4502cc6303f4aabd05e7fef6fbfaf4dfd1600bb6 (diff) | |
| download | chromium_src-ebff093d22cd5c0613f3493acdbc1af1cfd5d31f.zip chromium_src-ebff093d22cd5c0613f3493acdbc1af1cfd5d31f.tar.gz chromium_src-ebff093d22cd5c0613f3493acdbc1af1cfd5d31f.tar.bz2 | |
dbus::ObjectManager: Add a match rule for properties before GetManagedObjects.
There is a race condition in the way that match rules get set up for object
proxies created in response to GetManagedObjects that may cause us the miss
PropertiesChanged signals if they're received before the match rule and filter
function get added by ObjectProxy.
This patch changes this to work the "intended" way: ObjectManager now adds a
single match rule for its corresponding service name, and specifically for the
org.freedesktop.DBus.Properties.PropertiesChanged signal. Once it receives the
signal, ObjectManager dispatches the signal to the corresponding PropertySet.
BUG=407109,400768
TEST=dbus_unittests
Review URL: https://codereview.chromium.org/510863002
Cr-Commit-Position: refs/heads/master@{#293551}
Diffstat (limited to 'dbus/dbus.gyp')
| -rw-r--r-- | dbus/dbus.gyp | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/dbus/dbus.gyp b/dbus/dbus.gyp index 4d90497..3882df8 100644 --- a/dbus/dbus.gyp +++ b/dbus/dbus.gyp @@ -44,6 +44,8 @@ 'scoped_dbus_error.h', 'string_util.cc', 'string_util.h', + 'util.cc', + 'util.h', 'values_util.cc', 'values_util.h', ], |