dbus: Fix ResponseSender crash caused by lifetime mismatch
`ExportedObject` could out live its method call callbacks.
In such case, method call callbacks is no op hence not running
ResponseSender and crash. This CL checks the edge case and
send back `DBUS_ERROR_UNKNOWN_METHOD` error response.
Find a simpler way to export method and use it to update
existing `NotSendingResponseCrash` as well as adding a new
`SendFailureForShortLivedObject` to cover the fix. And reverted
the previous changes in `TestService`
Change ResponseSender CHECK to LOG(FATAL) to make it easier
to debug crashes that are not uploaded (hence no crash keys).
Re-enabled the disabled crosier bluetoothe tests.
Cq-Include-Trybots: luci.chrome.try:chromeos-trogdor-chrome
Bug: 404601483, 400758194
Change-Id: I578a7b2e902b71351104ce3ec0195ed16920202b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6373547
Commit-Queue: Xiyuan Xia <xiyuan@chromium.org>
Reviewed-by: Ryo Hashimoto <hashimoto@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1436057}
NOKEYCHECK=True
GitOrigin-RevId: 70afa29ed2e9b641773bfa6c203ced4bc5aa0e85
5 files changed