summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Sally Qi <sallyqi@google.com> 2024-09-11 00:58:22 +0000
committer Sally Qi <sallyqi@google.com> 2024-09-11 00:58:22 +0000
commit093477ce380eae99d41e077f531b22429a9889d0 (patch)
tree6398001049c9c9891fc541be876e41e20fc8bd54
parent5350e75cf32ed17d132f59c585b8ba282489b405 (diff)
[FrameTimeline] Don't add a skipped frame if the surface frame's jank type is none.
Bug: 316171339 Change-Id: I19b986c91de092c8b36f279f4714e1201c95c717 Test: perfetto checks Flag: EXEMPT bugfix
-rw-r--r--services/surfaceflinger/FrameTimeline/FrameTimeline.cpp2
1 files changed, 1 insertions, 1 deletions
diff --git a/services/surfaceflinger/FrameTimeline/FrameTimeline.cpp b/services/surfaceflinger/FrameTimeline/FrameTimeline.cpp
index 3736abc79f..2311360ce9 100644
--- a/services/surfaceflinger/FrameTimeline/FrameTimeline.cpp
+++ b/services/surfaceflinger/FrameTimeline/FrameTimeline.cpp
@@ -1205,7 +1205,7 @@ void FrameTimeline::DisplayFrame::addSkippedFrame(pid_t surfaceFlingerPid, nsecs
(static_cast<float>(previousPredictionPresentTime) +
kThresh * static_cast<float>(mRenderRate.getPeriodNsecs())) &&
// sf skipped frame is not considered if app is self janked
- !surfaceFrame->isSelfJanky()) {
+ surfaceFrame->getJankType() != JankType::None && !surfaceFrame->isSelfJanky()) {
skippedFrameStartTime = surfaceFrame->getPredictions().endTime;
skippedFramePresentTime = surfaceFrame->getPredictions().presentTime;
break;