diff options
| author | keybuk@chromium.org <keybuk@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-11-21 23:44:42 +0000 |
|---|---|---|
| committer | keybuk@chromium.org <keybuk@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-11-21 23:44:42 +0000 |
| commit | 66505bf8df02bb5e13843cd941f98de12621c62e (patch) | |
| tree | dcfb5671f38f906cbb446eba0a4c2e84c893b7f4 /dbus/object_proxy.cc | |
| parent | 7b7284b5bff831a811eaca8fae460143ed0a4dc5 (diff) | |
| download | chromium_src-66505bf8df02bb5e13843cd941f98de12621c62e.zip chromium_src-66505bf8df02bb5e13843cd941f98de12621c62e.tar.gz chromium_src-66505bf8df02bb5e13843cd941f98de12621c62e.tar.bz2 | |
Allow multiple object proxies to handle NameOwnerChanged
Multiple object proxies may be registered for any single D-Bus service, one per interface. Returning "handled" means only the first interface object proxy gets to handle the signal, others never see it.
Change to always return "not yet handled", while this means D-Bus never knows we actually handled the NameOwnerChanged signal, that's quite ok and is actually what all other bindings do for all signals.
BUG=chromium-os:36486
TEST=dbus_unittests
Review URL: https://chromiumcodereview.appspot.com/11280073
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@169164 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'dbus/object_proxy.cc')
| -rw-r--r-- | dbus/object_proxy.cc | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/dbus/object_proxy.cc b/dbus/object_proxy.cc index 87950d1..b20f544 100644 --- a/dbus/object_proxy.cc +++ b/dbus/object_proxy.cc @@ -661,11 +661,11 @@ DBusHandlerResult ObjectProxy::HandleNameOwnerChanged( name_owner_changed_callback_, released_signal)); } - return DBUS_HANDLER_RESULT_HANDLED; } } - // Untrusted or uninteresting signal + // Always return unhandled to let other object proxies handle the same + // signal. return DBUS_HANDLER_RESULT_NOT_YET_HANDLED; } |