tree e451d4ab118f0886e063bcd51766e121ce19f3ba
parent 9f0f77a7b373727ce1608f32d3ac0d1692821ab1
author armansito <armansito@chromium.org> 1409939374 +0900
committer Qijiang Fan <fqj@google.com> 1591259369 +0900

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}


CrOS-Libchrome-Original-Commit: ebff093d22cd5c0613f3493acdbc1af1cfd5d31f