diff options
| author | 2021-12-11 05:13:45 +0000 | |
|---|---|---|
| committer | 2021-12-11 06:28:25 +0000 | |
| commit | 3521e22b355fd95922715d70ec9be48dde5d2687 (patch) | |
| tree | 4394aa3e2e17ed39850db4231a0df18917fdc193 | |
| parent | 559dbb9079c13b3b479e7aa042a193685184998a (diff) | |
Add IME switcher button logging
Bug: 202885124
Test: statsd_testdrive -e -terse 90 (verify event shows after interaction)
Change-Id: Id5f6e13c5b14cf53c4154db39bb89b825cc31c15
3 files changed, 10 insertions, 0 deletions
diff --git a/packages/SystemUI/src/com/android/systemui/navigationbar/NavigationBar.java b/packages/SystemUI/src/com/android/systemui/navigationbar/NavigationBar.java index 03bceacdfbdc..7a12ecc097a2 100644 --- a/packages/SystemUI/src/com/android/systemui/navigationbar/NavigationBar.java +++ b/packages/SystemUI/src/com/android/systemui/navigationbar/NavigationBar.java @@ -1252,6 +1252,7 @@ public class NavigationBar implements View.OnAttachStateChangeListener, private void onImeSwitcherClick(View v) { mInputMethodManager.showInputMethodPickerFromSystem( true /* showAuxiliarySubtypes */, mDisplayId); + mUiEventLogger.log(KeyButtonView.NavBarButtonEvent.NAVBAR_IME_SWITCHER_BUTTON_TAP); }; private boolean onLongPressBackHome(View v) { diff --git a/packages/SystemUI/src/com/android/systemui/navigationbar/buttons/KeyButtonView.java b/packages/SystemUI/src/com/android/systemui/navigationbar/buttons/KeyButtonView.java index debd2ebb51be..d27b71673ce5 100644 --- a/packages/SystemUI/src/com/android/systemui/navigationbar/buttons/KeyButtonView.java +++ b/packages/SystemUI/src/com/android/systemui/navigationbar/buttons/KeyButtonView.java @@ -96,6 +96,9 @@ public class KeyButtonView extends ImageView implements ButtonInterface { @UiEvent(doc = "The overview button was pressed in the navigation bar.") NAVBAR_OVERVIEW_BUTTON_TAP(535), + @UiEvent(doc = "The ime switcher button was pressed in the navigation bar.") + NAVBAR_IME_SWITCHER_BUTTON_TAP(923), + @UiEvent(doc = "The home button was long-pressed in the navigation bar.") NAVBAR_HOME_BUTTON_LONGPRESS(536), diff --git a/packages/SystemUI/src/com/android/systemui/recents/OverviewProxyService.java b/packages/SystemUI/src/com/android/systemui/recents/OverviewProxyService.java index 3ed7e84af020..e7cd1e2dab3c 100644 --- a/packages/SystemUI/src/com/android/systemui/recents/OverviewProxyService.java +++ b/packages/SystemUI/src/com/android/systemui/recents/OverviewProxyService.java @@ -76,6 +76,7 @@ import androidx.annotation.NonNull; import com.android.internal.accessibility.dialog.AccessibilityButtonChooserActivity; import com.android.internal.annotations.VisibleForTesting; +import com.android.internal.logging.UiEventLogger; import com.android.internal.policy.ScreenDecorationsUtils; import com.android.internal.util.ScreenshotHelper; import com.android.systemui.Dumpable; @@ -88,6 +89,7 @@ import com.android.systemui.navigationbar.NavigationBar; import com.android.systemui.navigationbar.NavigationBarController; import com.android.systemui.navigationbar.NavigationBarView; import com.android.systemui.navigationbar.NavigationModeController; +import com.android.systemui.navigationbar.buttons.KeyButtonView; import com.android.systemui.recents.OverviewProxyService.OverviewProxyListener; import com.android.systemui.settings.CurrentUserTracker; import com.android.systemui.shared.recents.IOverviewProxy; @@ -161,6 +163,7 @@ public class OverviewProxyService extends CurrentUserTracker implements private final Optional<StartingSurface> mStartingSurface; private final SmartspaceTransitionController mSmartspaceTransitionController; private final Optional<RecentTasks> mRecentTasks; + private final UiEventLogger mUiEventLogger; private Region mActiveNavBarRegion; @@ -248,6 +251,7 @@ public class OverviewProxyService extends CurrentUserTracker implements mContext.getSystemService(InputMethodManager.class) .showInputMethodPickerFromSystem(true /* showAuxiliarySubtypes */, DEFAULT_DISPLAY); + mUiEventLogger.log(KeyButtonView.NavBarButtonEvent.NAVBAR_IME_SWITCHER_BUTTON_TAP); } @Override @@ -560,6 +564,7 @@ public class OverviewProxyService extends CurrentUserTracker implements ShellTransitions shellTransitions, ScreenLifecycle screenLifecycle, SmartspaceTransitionController smartspaceTransitionController, + UiEventLogger uiEventLogger, DumpManager dumpManager) { super(broadcastDispatcher); mContext = context; @@ -581,6 +586,7 @@ public class OverviewProxyService extends CurrentUserTracker implements mOneHandedOptional = oneHandedOptional; mShellTransitions = shellTransitions; mRecentTasks = recentTasks; + mUiEventLogger = uiEventLogger; // Assumes device always starts with back button until launcher tells it that it does not mNavBarButtonAlpha = 1.0f; |