tree 85f2148602cc5264a12097cd57062046e6d09f0d
parent 45735c6de3590b62918a73b93a1004f37d3dcfc3
author Sonny Sasaka <sonnysasaka@chromium.org> 1524678817 +0900
committer Qijiang Fan <fqj@google.com> 1591315828 +0900

dbus: Don't DCHECK unexpected message type, but ignore it

In dbus::ExportedObject, an assumption was made that no signal could be
received for an exported object, so DCHECK was put there. Now that there
is a use case of D-Bus forwarding by Bluetooth dispatcher, it is fine
that an exported object receives D-Bus signal messages as it is a
property change signal that is handled by dbus::ObjectManager. So we can
just ignore message types other than method in dbus::ExportedObject
rather than DCHECK it.

BUG=835457

Change-Id: Icae03881052b3d7365a9017e5f2b987a058cd356
Reviewed-on: https://chromium-review.googlesource.com/1027081
Reviewed-by: Ryo Hashimoto <hashimoto@chromium.org>
Reviewed-by: Satoru Takabayashi <satorux@google.com>
Commit-Queue: Sonny Sasaka <sonnysasaka@chromium.org>
Cr-Commit-Position: refs/heads/master@{#553639}

CrOS-Libchrome-Original-Commit: a3f2862214239a61f3c8557d80a17f26f84e2133