diff options
author | Jon Miranda <jonmiranda@google.com> | 2016-10-06 10:53:29 -0700 |
---|---|---|
committer | Jon Miranda <jonmiranda@google.com> | 2016-11-21 17:01:24 -0800 |
commit | feba90fe802cb54e02dd961dbea265c044ad5f9e (patch) | |
tree | fe4ac989638182de8c73f822141fa219b26d51d5 /src/com/android/launcher3/Launcher.java | |
parent | 0860e41d50ec2fbb16119fd364e36251b04932eb (diff) | |
download | android_packages_apps_Trebuchet-feba90fe802cb54e02dd961dbea265c044ad5f9e.tar.gz android_packages_apps_Trebuchet-feba90fe802cb54e02dd961dbea265c044ad5f9e.tar.bz2 android_packages_apps_Trebuchet-feba90fe802cb54e02dd961dbea265c044ad5f9e.zip |
Added logging for HOME_INTENT and BACK commands.
Document: go/false-longpress-overview
Change-Id: If6ba4996c861a44f4950b41acb4246e3a483d8f2
Diffstat (limited to 'src/com/android/launcher3/Launcher.java')
-rw-r--r-- | src/com/android/launcher3/Launcher.java | 34 |
1 files changed, 33 insertions, 1 deletions
diff --git a/src/com/android/launcher3/Launcher.java b/src/com/android/launcher3/Launcher.java index 9a5e1868a..9160a012e 100644 --- a/src/com/android/launcher3/Launcher.java +++ b/src/com/android/launcher3/Launcher.java @@ -1681,9 +1681,27 @@ public class Launcher extends Activity // Can be cases where mWorkspace is null, this prevents a NPE return; } - // In all these cases, only animate if we're already on home + + // Note: There should be at most one log per method call. This is enforced implicitly + // by using if-else statements. + UserEventDispatcher ued = getUserEventDispatcher(); + + // TODO: Log this case. mWorkspace.exitWidgetResizeMode(); + AbstractFloatingView topOpenView = AbstractFloatingView.getTopOpenView(this); + if (topOpenView instanceof DeepShortcutsContainer) { + ued.logActionCommand(LauncherLogProto.Action.HOME_INTENT, + topOpenView.getExtendedTouchView(), LauncherLogProto.DEEPSHORTCUTS); + } else if (topOpenView instanceof Folder) { + ued.logActionCommand(LauncherLogProto.Action.HOME_INTENT, + ((Folder) topOpenView).getFolderIcon(), LauncherLogProto.FOLDER); + } else if (alreadyOnHome) { + ued.logActionCommand(LauncherLogProto.Action.HOME_INTENT, + mWorkspace.getState().containerType, mWorkspace.getCurrentPage()); + } + + // In all these cases, only animate if we're already on home AbstractFloatingView.closeAllOpenViews(this, alreadyOnHome); exitSpringLoadedDragMode(); @@ -2187,20 +2205,34 @@ public class Launcher extends Activity return; } + // Note: There should be at most one log per method call. This is enforced implicitly + // by using if-else statements. + UserEventDispatcher ued = getUserEventDispatcher(); AbstractFloatingView topView = AbstractFloatingView.getTopOpenView(this); if (topView != null) { if (topView.getActiveTextView() != null) { topView.getActiveTextView().dispatchBackKey(); } else { + if (topView instanceof DeepShortcutsContainer) { + ued.logActionCommand(LauncherLogProto.Action.BACK, + topView.getExtendedTouchView(), LauncherLogProto.DEEPSHORTCUTS); + } else if (topView instanceof Folder) { + ued.logActionCommand(LauncherLogProto.Action.BACK, + ((Folder) topView).getFolderIcon(), LauncherLogProto.FOLDER); + } topView.close(true); } } else if (isAppsViewVisible()) { + ued.logActionCommand(LauncherLogProto.Action.BACK, LauncherLogProto.ALLAPPS); showWorkspace(true); } else if (isWidgetsViewVisible()) { + ued.logActionCommand(LauncherLogProto.Action.BACK, LauncherLogProto.WIDGETS); showOverviewMode(true); } else if (mWorkspace.isInOverviewMode()) { + ued.logActionCommand(LauncherLogProto.Action.BACK, LauncherLogProto.OVERVIEW); showWorkspace(true); } else { + // TODO: Log this case. mWorkspace.exitWidgetResizeMode(); // Back button is a no-op here, but give at least some feedback for the button press |