diff options
| author | 2023-10-29 13:27:46 -0700 | |
|---|---|---|
| committer | 2023-10-30 14:16:23 +0000 | |
| commit | 23d73fb740b943cc32b3336c873d4d79a42af858 (patch) | |
| tree | 3381cbdb84e2debfe57202db8bfe08d97bc2710e /libs/gui/FrameRateUtils.cpp | |
| parent | f217e6a37d98bae87e253a8d9706e510fa0a2a8a (diff) | |
Downgrade fatal checks to warning
These checks were previously ALOG_ASSERT, which is actually a non-fatal
assertion that does not cause a crash.
In ag/If2eae87bc2a40b61144ddcd019a9800c2d526072, the function
addWindowTargetLocked was forked into the pointer and non-pointer
versions:
1) addWindowTargetLocked (non-pointer)
2) addPointerWindowTargetLocked (for pointers specifically)
At the same time, these checks were converted into LOG_ALWAYS_FATAL.
This has helped during development of that CL, because it found several
paths where dispatch modes were getting altered unexpectedly.
Unfortunately, we are now receiving reports of crashes in these areas.
That means that our current test coverage is insufficient.
In this CL, the checks are getting downgraded to warnings. This will
help us focus on finding ways to reproducing the crashes that we
received so far. In the future, we can look into strengthening these
checks again.
Bug: 308302190
Test: TEST=inputflinger_tests; m $TEST && $ANDROID_HOST_OUT/nativetest64/$TEST/$TEST
Change-Id: I70f92d5576d733a1ba423115026c4ff66da8e682
Diffstat (limited to 'libs/gui/FrameRateUtils.cpp')
0 files changed, 0 insertions, 0 deletions