diff options
| author | 2024-09-11 00:58:22 +0000 | |
|---|---|---|
| committer | 2024-09-11 00:58:22 +0000 | |
| commit | 093477ce380eae99d41e077f531b22429a9889d0 (patch) | |
| tree | 6398001049c9c9891fc541be876e41e20fc8bd54 | |
| parent | 5350e75cf32ed17d132f59c585b8ba282489b405 (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.cpp | 2 |
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; |