Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Skip to content
Commit 72090cbd authored by Linnan Li's avatar Linnan Li Committed by Cherrypicker Worker
Browse files

Move input event verify for publish to back of sendMessage



Currently, in InputTransport, we support verifying events intended to
be published. This verification occurs before sendMessage. If the
application processes events too slowly at this time, it can lead to an
excessive number of events in the pipeline, preventing further sending.
In such cases, we return a status to the dispatcher to allow it to send
the event later. If the action of this event is DOWN or UP, the
InputVerifier will receive multiple events with action DOWN or UP,
which can cause problems with event verification, leading to abnormal
device crashes.

Here, we move the verification of the event to after sendMessage
succeeds. This resolves the issue because we only verify events that
have been successfully sent. Although this may cause the original logic
to change (potentially sending an incorrect stream of events to the
application), in reality, when an error occurs, the device has already
rebooted, so there is no impact.

Bug: 373764868
Flag: EXEMPT bugfix
Test: presubmit

Signed-off-by: default avatarLinnan Li <lilinnan@xiaomi.corp-partner.google.com>
(cherry picked from https://partner-android-review.googlesource.com/q/commit:586c7ba1af39fd81190e977fd8ecc570144d37da)
Merged-In: I74003e5d0eece9007db09cd98a60f616e0e7e22a
Change-Id: I74003e5d0eece9007db09cd98a60f616e0e7e22a
parent c17f5671
Loading
Loading
Loading
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment