commit 36b01af69dc5ef82befa80e3f4a01f7a955aa266
author Adithya Srinivasan <adsrini@google.com> Wed Apr 07 22:29:47 2021 +0000
committer Adithya Srinivasan <adsrini@google.com> Mon Apr 12 21:41:32 2021 +0000
tree 81b5a7b4ab67c9023bb352984743a64ca6ae34e6
parent 82eef326561d8eb2d668f5fd4144941795a1882d

Consider GPU comp info for jank classification

We now have gpu composition information in FrameTimeline. This change
updates the jank classification to consider this case as well.

Bug: 169876734
Test: libsurfaceflinger_unittest
Change-Id: I7bae2f9db52318f717196cf6033a1d2bb3596830

services/surfaceflinger/SurfaceFlinger.cpp

diff --git a/services/surfaceflinger/SurfaceFlinger.cpp b/services/surfaceflinger/SurfaceFlinger.cpp
index cbf8db8..41f211f 100644
--- a/services/surfaceflinger/SurfaceFlinger.cpp
+++ b/services/surfaceflinger/SurfaceFlinger.cpp
@@ -2173,7 +2173,7 @@
     // information from previous' frame classification is already available when sending jank info
     // to clients, so they get jank classification as early as possible.
     mFrameTimeline->setSfPresent(systemTime(), mPreviousPresentFences[0].fenceTime,
-                                 glCompositionDoneFenceTime != FenceTime::NO_FENCE);
+                                 glCompositionDoneFenceTime);
 
     nsecs_t dequeueReadyTime = systemTime();
     for (const auto& layer : mLayersWithQueuedFrames) {