diff options
author | Hyunyoung Song <hyunyoungs@google.com> | 2019-04-25 11:20:42 -0700 |
---|---|---|
committer | Hyunyoung Song <hyunyoungs@google.com> | 2019-04-25 11:56:04 -0700 |
commit | fd94cd54987931499e7252d03a5ecf6e1056dba0 (patch) | |
tree | fe37feb18ff40f8c9f28f178a3b80171a410f686 | |
parent | 01b8b68db77a22004168dd302ea1a4b2a5043b78 (diff) | |
download | android_packages_apps_Trebuchet-fd94cd54987931499e7252d03a5ecf6e1056dba0.tar.gz android_packages_apps_Trebuchet-fd94cd54987931499e7252d03a5ecf6e1056dba0.tar.bz2 android_packages_apps_Trebuchet-fd94cd54987931499e7252d03a5ecf6e1056dba0.zip |
Add logging for Task Icon interaction
Bug: 129279637
Change-Id: I135bad16015129b3b60bb13e86783d623e504b1b
-rw-r--r-- | protos/launcher_log.proto | 2 | ||||
-rw-r--r-- | quickstep/recents_ui_overrides/src/com/android/quickstep/views/TaskView.java | 10 |
2 files changed, 9 insertions, 3 deletions
diff --git a/protos/launcher_log.proto b/protos/launcher_log.proto index 4974dcbdc..02c6b0f41 100644 --- a/protos/launcher_log.proto +++ b/protos/launcher_log.proto @@ -71,6 +71,7 @@ enum ItemType { NOTIFICATION = 8; TASK = 9; // Each page of Recents UI (QuickStep) WEB_APP = 10; + TASK_ICON = 11; } // Used to define what type of container a Target would represent. @@ -115,6 +116,7 @@ enum ControlType { REMOTE_ACTION_SHORTCUT = 17; APP_USAGE_SETTINGS = 18; BACK_GESTURE = 19; + UNDO = 20; } enum TipType { diff --git a/quickstep/recents_ui_overrides/src/com/android/quickstep/views/TaskView.java b/quickstep/recents_ui_overrides/src/com/android/quickstep/views/TaskView.java index 05fe2d0d1..848c2140f 100644 --- a/quickstep/recents_ui_overrides/src/com/android/quickstep/views/TaskView.java +++ b/quickstep/recents_ui_overrides/src/com/android/quickstep/views/TaskView.java @@ -49,6 +49,8 @@ import com.android.launcher3.BaseDraggingActivity; import com.android.launcher3.R; import com.android.launcher3.anim.AnimatorPlaybackController; import com.android.launcher3.anim.Interpolators; +import com.android.launcher3.logging.UserEventDispatcher; +import com.android.launcher3.userevent.nano.LauncherLogProto; import com.android.launcher3.userevent.nano.LauncherLogProto.Action.Direction; import com.android.launcher3.userevent.nano.LauncherLogProto.Action.Touch; import com.android.launcher3.util.PendingAnimation; @@ -366,9 +368,11 @@ public class TaskView extends FrameLayout implements PageCallbacks, Reusable { } } - private boolean showTaskMenu() { + private boolean showTaskMenu(int action) { getRecentsView().snapToPage(getRecentsView().indexOfChild(this)); mMenuView = TaskMenuView.showForTask(this); + UserEventDispatcher.newInstance(getContext()).logActionOnItem(action, Direction.NONE, + LauncherLogProto.ItemType.TASK_ICON); if (mMenuView != null) { mMenuView.addOnAttachStateChangeListener(mTaskMenuStateListener); } @@ -378,10 +382,10 @@ public class TaskView extends FrameLayout implements PageCallbacks, Reusable { private void setIcon(Drawable icon) { if (icon != null) { mIconView.setDrawable(icon); - mIconView.setOnClickListener(v -> showTaskMenu()); + mIconView.setOnClickListener(v -> showTaskMenu(Touch.TAP)); mIconView.setOnLongClickListener(v -> { requestDisallowInterceptTouchEvent(true); - return showTaskMenu(); + return showTaskMenu(Touch.LONGPRESS); }); } else { mIconView.setDrawable(null); |