commit d432a7c40dfa6c5498038ec652db54478df966c1
author Lloyd Pique <lpique@google.com> Fri Mar 23 16:05:31 2018 -0700
committer Lloyd Pique <lpique@google.com> Wed Mar 28 16:48:45 2018 -0700
tree a488d5cd5f6785cad4c976e1e8736d29b15c5bb9
parent 0c2de3681db029180ed13b02f090bfa1636bae1c

SF: Clean up updateTransformHint() processing.

While working on ag/3368935, an unintentional duplicate call was added
to Layer::updateTransform() from
SurfaceFlinger::handleTransactionLocked(). This patch removes the
duplicate call.

In examining the code, the update conditions also did not appear to be
set correctly. They have been adjusted so that a display change or a
significant layer change (such as a layer being added) will result in an
update.

Bug: 76223411
Test: No immediate issues observed on a Pixel XL.

Change-Id: I59220837b22ae1f76d7d9d376c1ebd24d8d14105

services/surfaceflinger/SurfaceFlinger.h

diff --git a/services/surfaceflinger/SurfaceFlinger.h b/services/surfaceflinger/SurfaceFlinger.h
index 333dcbc..4322050 100644
--- a/services/surfaceflinger/SurfaceFlinger.h
+++ b/services/surfaceflinger/SurfaceFlinger.h
@@ -115,7 +115,8 @@
     eTransactionNeeded        = 0x01,
     eTraversalNeeded          = 0x02,
     eDisplayTransactionNeeded = 0x04,
-    eTransactionMask          = 0x07
+    eDisplayLayerStackChanged = 0x08,
+    eTransactionMask          = 0x0f,
 };
 
 class SurfaceFlingerBE