summaryrefslogtreecommitdiff
path: root/libs/gui/BufferQueueThreadState.cpp
diff options
context:
space:
mode:
author Jorim Jaggi <jjaggi@google.com> 2019-02-28 23:14:29 +0100
committer Jorim Jaggi <jjaggi@google.com> 2019-03-01 00:19:45 +0100
commitafa188d037bd5494347e6fd16245766b4499322e (patch)
tree90caf16d3394ea189fd5ea32246d25e33c97bd3b /libs/gui/BufferQueueThreadState.cpp
parentf1ece6cb2cb102168acf9a17c6ffc813905755b0 (diff)
Get rid of logic that clears sync point when layer is hidden
For app transitions that go to Launcher, Launcher is initially hidden, including the leash controlling it as well as the actual WSA surface. Since the barrier layer of the leash of the closing app is Launcher WSA, which is a child of the Launcher leash, the sync point will be cleared because it's not yet shown, as the showing itself is synced to WSA Launcher surface. However that destroys the synchronization, leading to jank in various ways. The justification why this code was added doesn't make sense to me: SurfaceView usually synchronizes with Activity main window surfaces. These get destroyed as soon as the activity goes away. We don't think that logic is needed at all. Test: Open app, press back, observe no jank Test: Play video in app Fixes: 126340236 Change-Id: Ib70d3bbdce1c7206487359213803e3e59d332400
Diffstat (limited to 'libs/gui/BufferQueueThreadState.cpp')
0 files changed, 0 insertions, 0 deletions