Avoid marking visible regions as dirty when metadata changes
This will trigger additional work by the main thread. This is a quick
fix before cleaning up layer state sequence ids.
Test: systrace and check we are not recomputing bounds when there
are no geometry changes
Fixes: 181369288
Change-Id: I6accbc6f6463f816c42e6a000900f15de982eb4b
diff --git a/services/surfaceflinger/Layer.cpp b/services/surfaceflinger/Layer.cpp
index 937868a..061ad0e 100644
--- a/services/surfaceflinger/Layer.cpp
+++ b/services/surfaceflinger/Layer.cpp
@@ -1339,7 +1339,6 @@
bool Layer::setMetadata(const LayerMetadata& data) {
if (!mCurrentState.metadata.merge(data, true /* eraseEmpty */)) return false;
- mCurrentState.sequence++;
mCurrentState.modified = true;
setTransactionFlags(eTransactionNeeded);
return true;