diff options
| author | Miranda Kephart <mkephart@google.com> | 2021-11-04 13:49:04 -0400 |
|---|---|---|
| committer | Miranda Kephart <mkephart@google.com> | 2021-11-05 13:55:13 +0000 |
| commit | 8e5f64b60b1151350814c0e30653e7a598514009 (patch) | |
| tree | d48073a28bfbc033a402f87a06152b3d933b1270 /tests/DynamicCodeLoggerIntegrationTests/src/cpp/test_executable.cpp | |
| parent | 0759523698149b65564f497194ed0c0e0a53ac0c (diff) | |
Ensure screenshot input monitor is always removed
We currently remove and dispose of the input monitor in View.onDetach.
However, it's possible for the screenshot process to get frozen
before that call actually occurs, which causes an ANR when the
screenshot process doesn't actually respond to input events. This
change adds a synchronous call to removeWindow to ensure that
the input monitor is never left around once the window is
removed.
Bug: 199592789
Test: manual -- forced a delay in detaching the view and stopping
the input listener, verified that the expected crash occurred,
and checked that adding the synchronous call fixes the problem
Change-Id: I0aff68ce4876082a5e20e3fe5f83cffcda4f6390
Merged-In: I0aff68ce4876082a5e20e3fe5f83cffcda4f6390
(cherry picked from commit 02de5054c5691e9425305a73f6210ca2b1cef499)
Diffstat (limited to 'tests/DynamicCodeLoggerIntegrationTests/src/cpp/test_executable.cpp')
0 files changed, 0 insertions, 0 deletions