From 98f4fc00ac8f6ca8289c8823b07bcc400576ef09 Mon Sep 17 00:00:00 2001 From: Vadim Tryshev Date: Mon, 14 May 2018 11:52:21 -0700 Subject: [PATCH] Following correct pattern for using logToggleRecents It supposed to involve checking enabledness. Bug: 72967764 Test: No Change-Id: I2a852932c665c24b8dc7c65f63bdaea91fa7ee5f --- .../src/com/android/quickstep/OverviewCommandHelper.java | 6 ++++-- .../com/android/quickstep/WindowTransformSwipeHandler.java | 3 +-- 2 files changed, 5 insertions(+), 4 deletions(-) diff --git a/quickstep/src/com/android/quickstep/OverviewCommandHelper.java b/quickstep/src/com/android/quickstep/OverviewCommandHelper.java index 20285016d7..c1462460b2 100644 --- a/quickstep/src/com/android/quickstep/OverviewCommandHelper.java +++ b/quickstep/src/com/android/quickstep/OverviewCommandHelper.java @@ -275,8 +275,10 @@ public class OverviewCommandHelper { } private AnimatorSet createWindowAnimation(RemoteAnimationTargetCompat[] targetCompats) { - LatencyTrackerCompat.logToggleRecents( - (int) (SystemClock.uptimeMillis() - mToggleClickedTime)); + if (LatencyTrackerCompat.isEnabled(mContext)) { + LatencyTrackerCompat.logToggleRecents( + (int) (SystemClock.uptimeMillis() - mToggleClickedTime)); + } if (mListener != null) { mListener.unregister(); diff --git a/quickstep/src/com/android/quickstep/WindowTransformSwipeHandler.java b/quickstep/src/com/android/quickstep/WindowTransformSwipeHandler.java index f29cc2257b..911a6028a6 100644 --- a/quickstep/src/com/android/quickstep/WindowTransformSwipeHandler.java +++ b/quickstep/src/com/android/quickstep/WindowTransformSwipeHandler.java @@ -435,9 +435,8 @@ public class WindowTransformSwipeHandler { mLayoutListener.setHandler(this); buildAnimationController(); - final long transitionDelay = mLauncherFrameDrawnTime - mTouchTimeMs; if (LatencyTrackerCompat.isEnabled(mContext)) { - LatencyTrackerCompat.logToggleRecents((int) transitionDelay); + LatencyTrackerCompat.logToggleRecents((int) (mLauncherFrameDrawnTime - mTouchTimeMs)); } }