From 9d38a3fd102a49cfd954876b57c75e391a09782e Mon Sep 17 00:00:00 2001 From: Brandon Dayauon Date: Tue, 27 Dec 2022 14:17:57 -0800 Subject: [PATCH] Add logging to work mode switch Add logging to the work button on work profile, logging when button is extended and collapsed bug: 263817965 test: presubmit Change-Id: I3194d0a1567a9ee995497391350ed5be4b08c3db --- .../launcher3/allapps/BaseAllAppsContainerView.java | 3 ++- src/com/android/launcher3/allapps/WorkModeSwitch.java | 8 ++++++++ .../android/launcher3/allapps/WorkProfileManager.java | 11 ++++++----- .../android/launcher3/logging/StatsLogManager.java | 8 +++++++- 4 files changed, 23 insertions(+), 7 deletions(-) diff --git a/src/com/android/launcher3/allapps/BaseAllAppsContainerView.java b/src/com/android/launcher3/allapps/BaseAllAppsContainerView.java index ca08164a05..00e89bacc5 100644 --- a/src/com/android/launcher3/allapps/BaseAllAppsContainerView.java +++ b/src/com/android/launcher3/allapps/BaseAllAppsContainerView.java @@ -163,7 +163,8 @@ public abstract class BaseAllAppsContainerView mAllApps; private final Predicate mMatcher; + private final StatsLogManager mStatsLogManager; private WorkModeSwitch mWorkModeSwitch; @@ -88,11 +89,13 @@ public class WorkProfileManager implements PersonalWorkSlidingTabStrip.OnActiveP private SharedPreferences mPreferences; public WorkProfileManager( - UserManager userManager, BaseAllAppsContainerView allApps, SharedPreferences prefs) { + UserManager userManager, BaseAllAppsContainerView allApps, SharedPreferences prefs, + StatsLogManager statsLogManager) { mUserManager = userManager; mAllApps = allApps; mPreferences = prefs; mMatcher = mAllApps.mPersonalMatcher.negate(); + mStatsLogManager = statsLogManager; } /** @@ -227,9 +230,7 @@ public class WorkProfileManager implements PersonalWorkSlidingTabStrip.OnActiveP private void onWorkFabClicked(View view) { if (Utilities.ATLEAST_P && mCurrentState == STATE_ENABLED && mWorkModeSwitch.isEnabled()) { - ActivityContext activityContext = ActivityContext.lookupContext( - mWorkModeSwitch.getContext()); - activityContext.getStatsLogManager().logger().log(LAUNCHER_TURN_OFF_WORK_APPS_TAP); + mStatsLogManager.logger().log(LAUNCHER_TURN_OFF_WORK_APPS_TAP); setWorkProfileEnabled(false); } } diff --git a/src/com/android/launcher3/logging/StatsLogManager.java b/src/com/android/launcher3/logging/StatsLogManager.java index 24d8c9dc5e..2159c6b68f 100644 --- a/src/com/android/launcher3/logging/StatsLogManager.java +++ b/src/com/android/launcher3/logging/StatsLogManager.java @@ -624,7 +624,13 @@ public class StatsLogManager implements ResourceBasedOverride { LAUNCHER_KEYBOARD_SHORTCUT_SPLIT_RIGHT_BOTTOM(1232), @UiEvent(doc = "User has invoked split to left half with a keyboard shortcut.") - LAUNCHER_KEYBOARD_SHORTCUT_SPLIT_LEFT_TOP(1233) + LAUNCHER_KEYBOARD_SHORTCUT_SPLIT_LEFT_TOP(1233), + + @UiEvent(doc = "User has collapsed the work FAB button by swiping down") + LAUNCHER_WORK_FAB_BUTTON_COLLAPSE(1276), + + @UiEvent(doc = "User has collapsed the work FAB button by swiping up") + LAUNCHER_WORK_FAB_BUTTON_EXTEND(1277), ; // ADD MORE