commit bb1ad7673511cf07b555ffd97b7eb0f9e5a8e382
author Ady Abraham <adyabr@google.com> Wed Mar 27 18:31:28 2024 -0700
committer Ady Abraham <adyabr@google.com> Thu Mar 28 01:36:17 2024 +0000
tree 103742500161edfa0622cac23104310457d0d399
parent 9c7dcd3e6b5930738b7d1efec889d1d51ade9ba2

SF: ignore eAutoRefreshChanged with latch unsignaled

eAutoRefreshChanged should not gate latch unsignaled behavior
as it doesn't change geometry or other layers.

Bug: 331513837
Test: presubmit
Change-Id: Ibceec1e94738e07a9de20c6276278e79198466e6

services/surfaceflinger/Layer.cpp

diff --git a/services/surfaceflinger/Layer.cpp b/services/surfaceflinger/Layer.cpp
index cd2120e..aed57e7 100644
--- a/services/surfaceflinger/Layer.cpp
+++ b/services/surfaceflinger/Layer.cpp
@@ -3853,8 +3853,10 @@
     const uint64_t deniedFlags = layer_state_t::eProducerDisconnect | layer_state_t::eLayerChanged |
             layer_state_t::eRelativeLayerChanged | layer_state_t::eTransparentRegionChanged |
             layer_state_t::eFlagsChanged | layer_state_t::eBlurRegionsChanged |
-            layer_state_t::eLayerStackChanged | layer_state_t::eAutoRefreshChanged |
-            layer_state_t::eReparent;
+            layer_state_t::eLayerStackChanged | layer_state_t::eReparent |
+            (FlagManager::getInstance().latch_unsignaled_with_auto_refresh_changed()
+                     ? 0
+                     : layer_state_t::eAutoRefreshChanged);
 
     if ((s.what & requiredFlags) != requiredFlags) {
         ATRACE_FORMAT_INSTANT("%s: false [missing required flags 0x%" PRIx64 "]", __func__,