summaryrefslogtreecommitdiff
path: root/rs/java/android/renderscript/Program.java
diff options
context:
space:
mode:
authorSiarhei Vishniakou <svv@google.com>2021-03-17 23:49:15 +0000
committerSiarhei Vishniakou <svv@google.com>2021-03-22 07:59:18 +0000
commit5054a9d00990ce35eb5ea6e5c9748d001a87c55c (patch)
tree6f93aa53cdb6dcbebf70480538bdc0427e996e36 /rs/java/android/renderscript/Program.java
parent0467858710e42343f6e67b1e8d11345b0459cdfb (diff)
Notify InputEventAssigner about every frame
InputEventAssigner does not currently get notified about all frames. In the existing logic, only the cases for batched consumption would be counted as 'choreographer callback'. This misses several cases, such as: 1) CALLBACK_INPUT was never scheduled, but a frame is produced 2) CALLBACK_INPUT was scheduled, but no events were consumed However, it is critical that InputEventAssigner knows about all frames, because it needs to reset the 'down' state for proper frame attribution. At the same time, we refactor the logic in InputEventAssigner. In the updated logic, we will specifically store the event id with ACTION_DOWN only. In all other cases, we will use the current / latest input event. In this CL, we reset the 'down' state after the input event id is used for the current frame. A good proxy for that is the ThreadedRenderer::draw call, which in turns calls attachInfo.mViewRootImpl.getUpdatedFrameInfo(). Also, rename "onChoreographerCallback" since it's not exactly clear which callback this is referring to. In the new logic, this callback will happen from 'draw'/'onDraw' functions. Bug: 167947340 Test: add LOG_ALWAYS_FATAL crashes in LatencyTracker for duplicate 'reportTimeline' calls. The crashes are gone after this CL. Change-Id: I5fad6665d10e0594963209b138e9bd2bdf00ca38
Diffstat (limited to 'rs/java/android/renderscript/Program.java')
0 files changed, 0 insertions, 0 deletions