commit 2b1b429c49ff59027578d5f046cba4082adcc7dc
author Sally Qi <sallyqi@google.com> Tue Mar 14 16:47:39 2023 +0000
committer Sally Qi <sallyqi@google.com> Wed Mar 15 01:23:49 2023 +0000
tree db2f020555ad01be7998f19916f18e73860e3b7c
parent 95d9bdd837e31e9fc8cce5047a763240f99c5874

Skip transaction processing if the surface damage stays the same in Layer::setSurfaceDamageRegion.

Bug: 273143519
Test: builds
Change-Id: Id4df40e869c2735ab7049eb50f07b81f56024001

services/surfaceflinger/Layer.cpp

diff --git a/services/surfaceflinger/Layer.cpp b/services/surfaceflinger/Layer.cpp
index 0f2af2f..9521371 100644
--- a/services/surfaceflinger/Layer.cpp
+++ b/services/surfaceflinger/Layer.cpp
@@ -3168,6 +3168,7 @@
 }
 
 bool Layer::setSurfaceDamageRegion(const Region& surfaceDamage) {
+    if (mDrawingState.surfaceDamageRegion.hasSameRects(surfaceDamage)) return false;
     mDrawingState.surfaceDamageRegion = surfaceDamage;
     mDrawingState.modified = true;
     setTransactionFlags(eTransactionNeeded);