Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 093477ce authored by Sally Qi's avatar Sally Qi
Browse files

[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
parent 5350e75c

services/surfaceflinger/FrameTimeline/FrameTimeline.cpp

+1 −1
Original line number Diff line number Diff line
@@ -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;