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

1 files changed, 0 insertions, 1 deletions

diff --git a/services/surfaceflinger/Layer.cpp b/services/surfaceflinger/Layer.cpp
index 937868a413..061ad0e03e 100644
--- a/services/surfaceflinger/Layer.cpp
+++ b/services/surfaceflinger/Layer.cpp
@@ -1339,7 +1339,6 @@ bool Layer::setCrop_legacy(const Rect& crop) {
 
 bool Layer::setMetadata(const LayerMetadata& data) {
     if (!mCurrentState.metadata.merge(data, true /* eraseEmpty */)) return false;
-    mCurrentState.sequence++;
     mCurrentState.modified = true;
     setTransactionFlags(eTransactionNeeded);
     return true;