From cdc13fc4e8768c44a681c18f4a46fad407d35325 Mon Sep 17 00:00:00 2001 From: Brian Isganitis Date: Sat, 4 Mar 2023 00:21:41 +0000 Subject: [PATCH] Log when FLAG_STASHED_IN_APP_AUTO changes. Bug: 261177876 Test: wwdebug & wwlogcat Change-Id: If2d2a3d38f3128a60d551bca9f4cdf64f2bcb032 --- .../android/launcher3/taskbar/TaskbarActivityContext.java | 1 - .../android/launcher3/taskbar/TaskbarStashController.java | 7 +++++++ src/com/android/launcher3/logging/StatsLogManager.java | 6 ++++++ 3 files changed, 13 insertions(+), 1 deletion(-) diff --git a/quickstep/src/com/android/launcher3/taskbar/TaskbarActivityContext.java b/quickstep/src/com/android/launcher3/taskbar/TaskbarActivityContext.java index 5c4f3e190d..f631a438c0 100644 --- a/quickstep/src/com/android/launcher3/taskbar/TaskbarActivityContext.java +++ b/quickstep/src/com/android/launcher3/taskbar/TaskbarActivityContext.java @@ -316,7 +316,6 @@ public class TaskbarActivityContext extends BaseTaskbarContext { return mTransientTaskbarBounds; } - @VisibleForTesting @Override public StatsLogManager getStatsLogManager() { // Used to mock, can't mock a default interface method directly diff --git a/quickstep/src/com/android/launcher3/taskbar/TaskbarStashController.java b/quickstep/src/com/android/launcher3/taskbar/TaskbarStashController.java index 115db25b52..539048aa44 100644 --- a/quickstep/src/com/android/launcher3/taskbar/TaskbarStashController.java +++ b/quickstep/src/com/android/launcher3/taskbar/TaskbarStashController.java @@ -24,6 +24,8 @@ import static com.android.launcher3.anim.Interpolators.LINEAR; import static com.android.launcher3.config.FeatureFlags.FORCE_PERSISTENT_TASKBAR; import static com.android.launcher3.logging.StatsLogManager.LauncherEvent.LAUNCHER_TASKBAR_LONGPRESS_HIDE; import static com.android.launcher3.logging.StatsLogManager.LauncherEvent.LAUNCHER_TASKBAR_LONGPRESS_SHOW; +import static com.android.launcher3.logging.StatsLogManager.LauncherEvent.LAUNCHER_TRANSIENT_TASKBAR_HIDE; +import static com.android.launcher3.logging.StatsLogManager.LauncherEvent.LAUNCHER_TRANSIENT_TASKBAR_SHOW; import static com.android.launcher3.taskbar.Utilities.appendFlag; import static com.android.systemui.shared.system.QuickStepContract.SYSUI_STATE_BOUNCER_SHOWING; import static com.android.systemui.shared.system.QuickStepContract.SYSUI_STATE_IME_SHOWING; @@ -967,6 +969,11 @@ public class TaskbarStashController implements TaskbarControllers.LoggableTaskba mActivity.getStatsLogManager().logger().log(LAUNCHER_TASKBAR_LONGPRESS_SHOW); } } + if (hasAnyFlag(changedFlags, FLAG_STASHED_IN_APP_AUTO)) { + mActivity.getStatsLogManager().logger().log(hasAnyFlag(FLAG_STASHED_IN_APP_AUTO) + ? LAUNCHER_TRANSIENT_TASKBAR_HIDE + : LAUNCHER_TRANSIENT_TASKBAR_SHOW); + } } private void notifyStashChange(boolean visible, boolean stashed) { diff --git a/src/com/android/launcher3/logging/StatsLogManager.java b/src/com/android/launcher3/logging/StatsLogManager.java index 5f6df277f8..cf710da484 100644 --- a/src/com/android/launcher3/logging/StatsLogManager.java +++ b/src/com/android/launcher3/logging/StatsLogManager.java @@ -636,6 +636,12 @@ public class StatsLogManager implements ResourceBasedOverride { @UiEvent(doc = "User scrolled up on the search result page.") LAUNCHER_ALLAPPS_SEARCH_SCROLLED_UP(1286), + + @UiEvent(doc = "User or automatic timeout has hidden transient taskbar.") + LAUNCHER_TRANSIENT_TASKBAR_HIDE(1330), + + @UiEvent(doc = "User has swiped upwards from the gesture handle to show transient taskbar.") + LAUNCHER_TRANSIENT_TASKBAR_SHOW(1331), ; // ADD MORE