commit 2269a699fd10485c45411d83ca4bfeeb7d64ef03
author Sally Qi <sallyqi@google.com> Fri May 17 18:02:28 2024 -0700
committer Sally Qi <sallyqi@google.com> Mon May 20 19:49:49 2024 -0700
tree e93c561c1ba31357f9e515e0d29ce8e04623793c
parent dba72f5654f6cb489e500d96ce1317e083da85aa

Fix the logic of adding skipped frame.

Bug: n/a
Test: FrameTimelineTest
Change-Id: I21d51d76229701f1fe66982488b5eefedeb02c0c

services/surfaceflinger/FrameTimeline/FrameTimeline.cpp

diff --git a/services/surfaceflinger/FrameTimeline/FrameTimeline.cpp b/services/surfaceflinger/FrameTimeline/FrameTimeline.cpp
index 8369a1e..c8e1fbb 100644
--- a/services/surfaceflinger/FrameTimeline/FrameTimeline.cpp
+++ b/services/surfaceflinger/FrameTimeline/FrameTimeline.cpp
@@ -1166,7 +1166,7 @@
                     (static_cast<float>(mSurfaceFlingerPredictions.presentTime) -
                      kThresh * static_cast<float>(mRenderRate.getPeriodNsecs())) &&
             static_cast<float>(surfaceFrame->getPredictions().presentTime) >=
-                    (static_cast<float>(previousPredictionPresentTime) -
+                    (static_cast<float>(previousPredictionPresentTime) +
                      kThresh * static_cast<float>(mRenderRate.getPeriodNsecs())) &&
             // sf skipped frame is not considered if app is self janked
             !surfaceFrame->isSelfJanky()) {