commit 0449b0fa3e3e88da8622da6e95a7eefea8c46a70
author Lloyd Pique <lpique@google.com> Thu Dec 20 16:23:45 2018 -0800
committer Lloyd Pique <lpique@google.com> Mon Jan 07 13:39:45 2019 -0800
tree 57eac962d959508de399454604076cd343d36f66
parent f2c793939c42ff233ced4095cf85bb93ec74523f

Revert "SurfaceFlinger: protect state members in Layer"

State update transactions must be atomic. The fine-grained lock on each
Layer implied otherwise. Revert the fine grained lock as being
unhelpful.

Unfortunately there does not seem to be a way to use Clang thread
annotations to specify the desired locking behavior.

Note that the parent CL addresses the locking problem that led to the
bug.

This reverts commit 83729883eecd31a9907bc79bc21998a90f17105c.

Bug: 119481871
Test: SurfaceFlinger unit tests
Test: go/wm-smoke
Change-Id: I361741f8d10102aeb57f164c847c6063ff93dd14