diff options
| author | 2024-10-18 23:16:43 +0000 | |
|---|---|---|
| committer | 2024-10-25 01:51:53 +0000 | |
| commit | 37242c82244e56acf73458b83706d6b11e8b1289 (patch) | |
| tree | a2f6a9cb9d5759d49d24671670df221846c7bcf2 /libs/gui/BufferReleaseChannel.cpp | |
| parent | 1a1094a3b32f221d41a1740277ffaf766c62c15c (diff) | |
Do not resample when frameTime is not available
LegacyResampler still resamples MotionEvents even when requestedFrameTime is
std::nullopt. The problem is that if the parameter requestedFrameTime
is std::nullopt, it is reassigned inside consumeBatchedInputEvents to
std::numeric_limits<nsecs_t>::max(). In spite of using max() as a
placeholder to indicate that everything should be consumed, it is still a valid time, and resampler uses it to resample MotionEvent. To fix this issue, having a valid requestedFrameTime should also be a condition to resample MotionEvents.
Bug: 374375203
Flag: EXEMPT refactor
Test: TEST=libinput_tests; m $TEST && $ANDROID_HOST_OUT/nativetest64/$TEST/$TEST
Change-Id: I6e53bdd0a35d359da8b50f10dd4aad9bddc8aa3f
Diffstat (limited to 'libs/gui/BufferReleaseChannel.cpp')
0 files changed, 0 insertions, 0 deletions