commit 412c592e90b0c5d93d8d009f4b6f5a93a3adfaa2
author Vishnu Nair <vishnun@google.com> Thu Jul 01 12:05:02 2021 -0700
committer Vishnu Nair <vishnun@google.com> Thu Jul 01 12:05:23 2021 -0700
tree cba4530435ff1cf06486736cf06d7fe25471dc76
parent e1c1b1c55dfc8ec89fd1dd4bb1325bd3428a03a1

Don't set the visibleRegion dirty bit if blur regions are changing

Test: check winscope trace when pulling down notification shade, see frames are
treated as buffer latches
Bug: 191969790

Change-Id: Ifa73d30dcae6d9336cf8e3d709dd8766c9d1b591

services/surfaceflinger/Layer.cpp

diff --git a/services/surfaceflinger/Layer.cpp b/services/surfaceflinger/Layer.cpp
index ad31b3f..5dc0242 100644
--- a/services/surfaceflinger/Layer.cpp
+++ b/services/surfaceflinger/Layer.cpp
@@ -954,7 +954,6 @@
 }
 
 bool Layer::setBlurRegions(const std::vector<BlurRegion>& blurRegions) {
-    mDrawingState.sequence++;
     mDrawingState.blurRegions = blurRegions;
     mDrawingState.modified = true;
     setTransactionFlags(eTransactionNeeded);