commit 5e83a40f15b72f7ea3d73da34e6212a0ff0c6888
author Yilong Li <liyl@google.com> Tue Oct 08 19:04:11 2024 +0000
committer CQ Bot <fuchsia-internal-scoped@luci-project-accounts.iam.gserviceaccount.com> Tue Oct 08 19:04:11 2024 +0000
tree 9c30a51adb872d1b7cf6010593d2efb0d8357d3c
parent eb9e6b09067cbc336bd343201ad2b1279d16a498

[display][coordinator] Deregister the engine listener in `PrepareStop()`.

Previously, the display coordinator Controller deregistered it
from the display engine at the Controller dtor. This caused a
race between the coordinator and the engine driver at destruction.

This change moves the deregistration logic from the destructor to
the `PrepareStop()` method where both drivers are guaranteed to
be alive.

Change-Id: Ic3b53fe243ae26cf8c3404a93a0851f861733e7e
Test: dm_reboot_bootloader_test
Multiply: host_x64/obj/src/tests/reboot/dm_reboot_bootloader_test.sh: 100
Multiply: host_x64/obj/src/tests/reboot/dm_reboot_recovery_test.sh: 100
Multiply: host_x64/obj/src/tests/reboot/dm_reboot_test.sh: 100
Bug: 367330135
Reviewed-on: https://fuchsia-review.googlesource.com/c/fuchsia/+/1133172
Commit-Queue: Victor Costan <costan@google.com>
Fuchsia-Auto-Submit: Victor Costan <costan@google.com>
Reviewed-by: Victor Costan <costan@google.com>