From 0ee7ffca7f3b71c4b8eca6562ddddd48f03cca92 Mon Sep 17 00:00:00 2001 From: Marcin Oczeretko Date: Thu, 8 Oct 2020 16:05:08 +0100 Subject: [PATCH] Update LatencyTracker usage in Launcher Test: manual - flashed the phone and verified the logging and statsd sampling with default settings Bug: 131859447, 162927074 Change-Id: I2d0c9fd479654c62841c714eb39f88f84e8ab114 --- quickstep/src/com/android/quickstep/AbsSwipeUpHandler.java | 7 ++----- .../src/com/android/quickstep/OverviewCommandHelper.java | 6 ++---- 2 files changed, 4 insertions(+), 9 deletions(-) diff --git a/quickstep/src/com/android/quickstep/AbsSwipeUpHandler.java b/quickstep/src/com/android/quickstep/AbsSwipeUpHandler.java index ef7a8f6477..202aa0dd62 100644 --- a/quickstep/src/com/android/quickstep/AbsSwipeUpHandler.java +++ b/quickstep/src/com/android/quickstep/AbsSwipeUpHandler.java @@ -473,11 +473,8 @@ public abstract class AbsSwipeUpHandler, Q extends Object traceToken = TraceHelper.INSTANCE.beginSection("logToggleRecents", TraceHelper.FLAG_IGNORE_BINDERS); - // Only used in debug builds - if (LatencyTrackerCompat.isEnabled(mContext)) { - LatencyTrackerCompat.logToggleRecents( - (int) (mLauncherFrameDrawnTime - mTouchTimeMs)); - } + LatencyTrackerCompat.logToggleRecents( + mContext, (int) (mLauncherFrameDrawnTime - mTouchTimeMs)); TraceHelper.INSTANCE.endSection(traceToken); // This method is only called when STATE_GESTURE_STARTED is set, so we can enable the diff --git a/quickstep/src/com/android/quickstep/OverviewCommandHelper.java b/quickstep/src/com/android/quickstep/OverviewCommandHelper.java index 365cff576a..86136873ef 100644 --- a/quickstep/src/com/android/quickstep/OverviewCommandHelper.java +++ b/quickstep/src/com/android/quickstep/OverviewCommandHelper.java @@ -224,10 +224,8 @@ public class OverviewCommandHelper { private AnimatorSet createWindowAnimation(RemoteAnimationTargetCompat[] appTargets, RemoteAnimationTargetCompat[] wallpaperTargets) { - if (LatencyTrackerCompat.isEnabled(mContext)) { - LatencyTrackerCompat.logToggleRecents( - (int) (SystemClock.uptimeMillis() - mToggleClickedTime)); - } + LatencyTrackerCompat.logToggleRecents( + mContext, (int) (SystemClock.uptimeMillis() - mToggleClickedTime)); mListener.unregister();