diff options
| author | John Reck <jreck@google.com> | 2018-09-14 11:25:58 -0700 |
|---|---|---|
| committer | John Reck <jreck@google.com> | 2018-09-14 11:25:58 -0700 |
| commit | 50be3b83783b57492149b235ff220a7f1519327b (patch) | |
| tree | 1d8e1296670880cae532e938550cfc6451ed6d77 /libs/hwui/JankTracker.cpp | |
| parent | 0b8f17b3ba65d9c8c59600eaa2b0e30dc032e91f (diff) | |
Don't run comparisons on non-jank frames
Unclear why mSwapDeadline doesn't already
account for this, but the handling of vsync
phase offsets may be throwing things off a bit.
For now just do the simple fix.
Bug: 109894489
Test: builds, patch was verified by reporter
Change-Id: I5a6f25ceac4986d366293fe7b5e3af64a326114c
Diffstat (limited to 'libs/hwui/JankTracker.cpp')
| -rw-r--r-- | libs/hwui/JankTracker.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/libs/hwui/JankTracker.cpp b/libs/hwui/JankTracker.cpp index e6d2a6f555ac..f2d50cd42523 100644 --- a/libs/hwui/JankTracker.cpp +++ b/libs/hwui/JankTracker.cpp @@ -146,7 +146,7 @@ void JankTracker::finishFrame(const FrameInfo& frame) { frame[FrameInfoIndex::IntendedVsync] + mFrameInterval); // If we hit the deadline, cool! - if (frame[FrameInfoIndex::FrameCompleted] < mSwapDeadline) { + if (frame[FrameInfoIndex::FrameCompleted] < mSwapDeadline || totalDuration < mFrameInterval) { if (isTripleBuffered) { mData->reportJankType(JankType::kHighInputLatency); (*mGlobalData)->reportJankType(JankType::kHighInputLatency); |