Allow multiple object proxies to handle NameOwnerChanged

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.

Submitted as -r169164 but reverted in -r169170; resubmitted with change to run message loop three times if necessary.

BUG=chromium-os:36486
TEST=dbus_unittests

TBR=gauravsh@chromium.org
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=169538

[gauravsh, 2012-11-20 00:02:10]

lgtm from me - I have tested the fix and it does resolve crosbug.com/34399 in
face of shill restarts.

[mukesh agrawal, 2012-11-20 00:11:55]

lgtm from here, as well.


On Mon, Nov 19, 2012 at 4:02 PM, <gauravsh@chromium.org> wrote:

> lgtm from me - I have tested the fix and it does resolve crosbug.com/34399in
> face of shill restarts.
>
>
https://codereview.chromium.**org/11280073/<https://codereview.chromium.org/1...
>

[Haruki Sato, 2012-11-20 01:31:22]

lgtm

Thank you for the quick fix!

On 2012/11/20 00:11:55, mukesh agrawal wrote:
> lgtm from here, as well.
> 
> 
> On Mon, Nov 19, 2012 at 4:02 PM, <mailto:gauravsh@chromium.org> wrote:
> 
> > lgtm from me - I have tested the fix and it does resolve crosbug.com/34399in
> > face of shill restarts.
> >
> >
>
https://codereview.chromium.**org/11280073/%3Chttps://codereview.chromium.org...>
> >

[satorux1, 2012-11-20 05:01:49]

LGTM

[commit-bot: I haz the power, 2012-11-21 21:42:08]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/keybuk@chromium.org/11280073/1

[commit-bot: I haz the power, 2012-11-21 23:44:43]

Change committed as 169164