From 6f787a758808dd6084c24b76853f2dc2721e4caf Mon Sep 17 00:00:00 2001 From: Ahan Wu Date: Tue, 7 Sep 2021 09:18:48 -0800 Subject: [PATCH] Shorten the timeout of finish call to 10 seconds We have a minute timeout of finish call, it's a long duration, shorten it to 10 seconds. Bug: 196171638 Test: Manually Change-Id: I4ff726bbac13fd90e7578074b5f11170816be7b1 --- core/java/com/android/internal/jank/FrameTracker.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/core/java/com/android/internal/jank/FrameTracker.java b/core/java/com/android/internal/jank/FrameTracker.java index 66c0ce26560c0..c863292d4ea0a 100644 --- a/core/java/com/android/internal/jank/FrameTracker.java +++ b/core/java/com/android/internal/jank/FrameTracker.java @@ -288,12 +288,12 @@ public class FrameTracker extends SurfaceControl.OnJankDataListener // We don't remove observer here, // will remove it when all the frame metrics in this duration are called back. // See onFrameMetricsAvailable for the logic of removing the observer. - // Let's wait for all callbacks to finish for at most a minute. + // Waiting at most 10 seconds for all callbacks to finish. mWaitForFinishTimedOut = () -> { Log.e(TAG, "force finish cuj because of time out:" + mSession.getName()); finish(mJankInfos.size() - 1); }; - mHandler.postDelayed(mWaitForFinishTimedOut, TimeUnit.MINUTES.toMillis(1)); + mHandler.postDelayed(mWaitForFinishTimedOut, TimeUnit.SECONDS.toMillis(10)); } }