diff options
| author | chaviw <chaviw@google.com> | 2021-04-07 17:35:12 -0500 |
|---|---|---|
| committer | chaviw <chaviw@google.com> | 2021-04-09 10:35:03 -0500 |
| commit | f541d037f59dc178db81babf6f7fa84c9c3072d5 (patch) | |
| tree | 1aeb255935832c3c62850767428b90f54d5ad276 /tools/aapt2/java/JavaClassGenerator_test.cpp | |
| parent | d4fd1b0ce29875d2f533fe2ead4656fceca62ba9 (diff) | |
Use separate Transaction object in SV
Currently, setAlpha was using mRtTransaction to send the SC transaction.
However, that's also used by positionChanged. This could cause
synchronization issues because both callbacks are invoked from a thread
pool with no synchronization ensured.
Instead, create a Transaction that's used just for FrameCallbacks.
FrameCallbacks will all occur on the same thread so it's safe to use the
same object. Additionally, created a separate transaction that should be
used for positionChanged. This is because it's invoked from a thread
pool can could interfere with the main render thread transactions.
Finally, use the original mRtTransaction only for places that are called
synchronously from renderThread.
Test: Hard to repro original issue. Bubbles works
Bug: 184559328
Change-Id: Ia5a36ad063c19e8a481a57170e47029f7c2c7ee7
Diffstat (limited to 'tools/aapt2/java/JavaClassGenerator_test.cpp')
0 files changed, 0 insertions, 0 deletions