commit 0fe42e5bdfc5ec1a828c2d5d1ec04520cb652b62
author Lucas Dupin <dupin@google.com> Thu May 27 17:27:15 2021 -0700
committer Lucas Dupin <dupin@google.com> Tue Jun 01 12:22:27 2021 -0700
tree b4b4f4eb6d0700feed5962e6a2308347ce9c0352
parent f1804968780f975fa8fe361c480a15e8dd85d131

Mark empty EffectLayer as skipContentDraw

An effect layer that only draws a shadow or blurs shouldn't draw a
color. We should skip drawing its contents altogether.

Bug: 189207458
Test: manual
Change-Id: I01bb066d4a02f6111a6974dd241f9530c4979420

services/surfaceflinger/Layer.cpp

diff --git a/services/surfaceflinger/Layer.cpp b/services/surfaceflinger/Layer.cpp
index a7c8704..690fda6 100644
--- a/services/surfaceflinger/Layer.cpp
+++ b/services/surfaceflinger/Layer.cpp
@@ -574,7 +574,8 @@
     layerSettings.geometry.positionTransform = getTransform().asMatrix4();
 
     // skip drawing content if the targetSettings indicate the content will be occluded
-    layerSettings.skipContentDraw = !targetSettings.realContentIsVisible;
+    layerSettings.skipContentDraw =
+            layerSettings.skipContentDraw || !targetSettings.realContentIsVisible;
 
     if (hasColorTransform()) {
         layerSettings.colorTransform = getColorTransform();