From be352b589c98a8b58933bcbae2b99017bf30e45e Mon Sep 17 00:00:00 2001 From: Andy Wickham Date: Wed, 6 Nov 2024 12:21:11 -0800 Subject: [PATCH] Improve 2 stage logging slightly. The OtherActivityInputConsumer inverted log was inadvertently added in ag/28270874. Flag: EXEMPT logs only Bug: NA Test: Manual Change-Id: Iab5de4ec26bd40d18bdcdea62ec7ecb685fec6a1 --- .../inputconsumers/NavHandleLongPressInputConsumer.java | 2 +- .../inputconsumers/OtherActivityInputConsumer.java | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/quickstep/src/com/android/quickstep/inputconsumers/NavHandleLongPressInputConsumer.java b/quickstep/src/com/android/quickstep/inputconsumers/NavHandleLongPressInputConsumer.java index f5bef05e14..afe988d4eb 100644 --- a/quickstep/src/com/android/quickstep/inputconsumers/NavHandleLongPressInputConsumer.java +++ b/quickstep/src/com/android/quickstep/inputconsumers/NavHandleLongPressInputConsumer.java @@ -242,7 +242,7 @@ public class NavHandleLongPressInputConsumer extends DelegateInputConsumer { private void cancelLongPress(String reason) { if (DEBUG_NAV_HANDLE) { - Log.d(TAG, "cancelLongPress"); + Log.d(TAG, "cancelLongPress: " + reason); } mGestureState.setIsInExtendedSlopRegion(false); MAIN_EXECUTOR.getHandler().removeCallbacks(mTriggerLongPress); diff --git a/quickstep/src/com/android/quickstep/inputconsumers/OtherActivityInputConsumer.java b/quickstep/src/com/android/quickstep/inputconsumers/OtherActivityInputConsumer.java index e19b338dba..d63ddb39e5 100644 --- a/quickstep/src/com/android/quickstep/inputconsumers/OtherActivityInputConsumer.java +++ b/quickstep/src/com/android/quickstep/inputconsumers/OtherActivityInputConsumer.java @@ -293,15 +293,15 @@ public class OtherActivityInputConsumer extends ContextWrapper implements InputC float upDist = -displacement; boolean isTrackpadGesture = mGestureState.isTrackpadGesture(); float squaredHypot = squaredHypot(displacementX, displacementY); - boolean isInExtendedSlopRegion = !mGestureState.isInExtendedSlopRegion(); + boolean isInExtendedSlopRegion = mGestureState.isInExtendedSlopRegion(); boolean passedSlop = isTrackpadGesture || (squaredHypot >= mSquaredTouchSlop - && isInExtendedSlopRegion); + && !isInExtendedSlopRegion); if (DEBUG) { Log.d(TAG, "ACTION_MOVE: passedSlop=" + passedSlop + " ( " + isTrackpadGesture + " || (" + squaredHypot + " >= " + mSquaredTouchSlop - + " && " + isInExtendedSlopRegion + " ))"); + + " && " + !isInExtendedSlopRegion + " ))"); } if (!mPassedSlopOnThisGesture && passedSlop) {