diff options
| author | Sonny Sasaka <sonnysasaka@chromium.org> | 2018-05-07 23:52:27 -0700 |
|---|---|---|
| committer | chrome-bot <chrome-bot@chromium.org> | 2018-06-05 15:59:33 -0700 |
| commit | ce194b9773af8bdbf6f5c8f8ee244786a07952bb (patch) | |
| tree | 77f002be8d4c23445c28ebefc3b6b8ba227311e4 | |
| parent | d7220e5c8555ebae190362b554973e7458eff13c (diff) | |
| download | libbrillo-ce194b9773af8bdbf6f5c8f8ee244786a07952bb.tar.gz | |
dbus: Fix Properties.Set handler to also emit PropertiesChanged
When a D-Bus property value is changed, it should emit PropertiesChanged
signal.
BUG=chromium:840517
TEST=None
Change-Id: I414a62b04f4fb1383152375e1efae434c114e2be
Reviewed-on: https://chromium-review.googlesource.com/1049169
Commit-Ready: Sonny Sasaka <sonnysasaka@chromium.org>
Tested-by: Sonny Sasaka <sonnysasaka@chromium.org>
Reviewed-by: Dan Erat <derat@chromium.org>
Reviewed-by: Ben Chan <benchan@chromium.org>
| -rw-r--r-- | brillo/dbus/exported_property_set.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/brillo/dbus/exported_property_set.h b/brillo/dbus/exported_property_set.h index 61ea08a..971e932 100644 --- a/brillo/dbus/exported_property_set.h +++ b/brillo/dbus/exported_property_set.h @@ -215,7 +215,7 @@ class ExportedProperty : public ExportedPropertyBase { if (!validator_.is_null() && !validator_.Run(error, value.Get<T>())) { return false; } - value_ = value.Get<T>(); + SetValue(value.Get<T>()); return true; } |