Don't increment layer state sequence id when setting setFrameTimelineVsyncForTransaction
Layer state sequence id should only change when the visible region
can change. The id is used to set the mVisibleRegionDirty flag which
triggers geom calculations and updates to InputDispatcher.
setFrameTimelineVsyncForTransaction is called for every layer
transaction and this is worse for Blast since buffers come via
transactions as well.
Test: adb shell dumpsys SurfaceFlinger --frametimeline and check prediction state is valid
Change-Id: Ib8ba61a706d122e3e2d77730dcabf29c7480529b
diff --git a/services/surfaceflinger/Layer.cpp b/services/surfaceflinger/Layer.cpp
index 629bee8..50707df 100644
--- a/services/surfaceflinger/Layer.cpp
+++ b/services/surfaceflinger/Layer.cpp
@@ -1455,7 +1455,6 @@
}
void Layer::setFrameTimelineVsyncForTransaction(int64_t frameTimelineVsyncId, nsecs_t postTime) {
- mCurrentState.sequence++;
mCurrentState.frameTimelineVsyncId = frameTimelineVsyncId;
mCurrentState.postTime = postTime;
mCurrentState.modified = true;