From 392a265c1dd7d9602d9520f807ba95b723ff1471 Mon Sep 17 00:00:00 2001 From: helencheuk Date: Thu, 26 Sep 2024 15:34:48 +0100 Subject: [PATCH] [Contextual Edu] Not update edu stats when swiping down all apps panel It should only update edu stats of home gesture when user swipe up to go home. It is already handled in AbsSwipeUpHandler. This change avoids updating edu stats when users swipe down the all apps panel as it is not considered as home gesture. Bug: 363480554 Test: Manual Flag: com.android.systemui.keyboard_touchpad_contextual_education Change-Id: Ib859cef5cbb2805fd2b596c6c40a67a5f21cceae --- .../launcher3/touch/AbstractStateChangeTouchController.java | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/src/com/android/launcher3/touch/AbstractStateChangeTouchController.java b/src/com/android/launcher3/touch/AbstractStateChangeTouchController.java index efd1f0d6b4..74a09669a7 100644 --- a/src/com/android/launcher3/touch/AbstractStateChangeTouchController.java +++ b/src/com/android/launcher3/touch/AbstractStateChangeTouchController.java @@ -407,10 +407,7 @@ public abstract class AbstractStateChangeTouchController } private void updateContextualEduStats(LauncherState targetState) { - if (targetState == NORMAL) { - ContextualEduStatsManager.INSTANCE.get( - mLauncher).updateEduStats(mDetector.isTrackpadGesture(), GestureType.HOME); - } else if (targetState == OVERVIEW) { + if (targetState == OVERVIEW) { ContextualEduStatsManager.INSTANCE.get( mLauncher).updateEduStats(mDetector.isTrackpadGesture(), GestureType.OVERVIEW); } else if (targetState == ALL_APPS && !mDetector.isTrackpadGesture()) {