tree 48c383a3b955d4ed677996ce3cecedf4c5891da8
parent 43d32906a8a51b37bbb118c8febd4cea341331b4
author Robert Carr <racarr@google.com> 1589394223 -0700
committer Rob Carr <racarr@google.com> 1589394599 +0000

SurfaceView: positionLost locking fix

positionLost can be called from CanvasContext::destroyHardwareResources
which runs asynchronously to the UI thread. This means we could be
simultaneously executing releaseSurfaces on the UI thread. We need
to expand the scope of mSurfaceControl lock in positionLost. While
we are here we add a block comment explaining the previously
undocumented locking strategy.

Bug: 156264048
Test: Existing tests pass
Change-Id: I9cdb6a0f7aeffd878f1755f240e8896f0fb8bf01