Merge "Add CUJ-LAUNCHER_OPEN_ALL_APPS instrumentation from taskbar" into main
This commit is contained in:
committed by
Android (Google) Code Review
commit
e2a76cee2b
@@ -23,6 +23,9 @@ import android.view.InputDevice;
|
||||
import android.view.MotionEvent;
|
||||
import android.view.View;
|
||||
|
||||
import com.android.internal.jank.Cuj;
|
||||
import com.android.systemui.shared.system.InteractionJankMonitorWrapper;
|
||||
|
||||
/**
|
||||
* Callbacks for {@link TaskbarView} to interact with its controller.
|
||||
*/
|
||||
@@ -45,6 +48,8 @@ public class TaskbarViewCallbacks {
|
||||
|
||||
public View.OnClickListener getAllAppsButtonClickListener() {
|
||||
return v -> {
|
||||
InteractionJankMonitorWrapper.begin(v, Cuj.CUJ_LAUNCHER_OPEN_ALL_APPS,
|
||||
/* tag= */ "TASKBAR_BUTTON");
|
||||
mActivity.getStatsLogManager().logger().log(LAUNCHER_TASKBAR_ALLAPPS_BUTTON_TAP);
|
||||
mControllers.taskbarAllAppsController.toggle();
|
||||
};
|
||||
|
||||
@@ -20,6 +20,7 @@ import static com.android.launcher3.util.OnboardingPrefs.ALL_APPS_VISITED_COUNT;
|
||||
|
||||
import androidx.annotation.Nullable;
|
||||
|
||||
import com.android.internal.jank.Cuj;
|
||||
import com.android.launcher3.AbstractFloatingView;
|
||||
import com.android.launcher3.allapps.AllAppsTransitionListener;
|
||||
import com.android.launcher3.anim.PendingAnimation;
|
||||
@@ -31,6 +32,7 @@ import com.android.launcher3.taskbar.TaskbarStashController;
|
||||
import com.android.launcher3.taskbar.overlay.TaskbarOverlayContext;
|
||||
import com.android.launcher3.taskbar.overlay.TaskbarOverlayController;
|
||||
import com.android.launcher3.util.DisplayController;
|
||||
import com.android.systemui.shared.system.InteractionJankMonitorWrapper;
|
||||
|
||||
import java.util.Optional;
|
||||
|
||||
@@ -136,6 +138,9 @@ final class TaskbarAllAppsViewController {
|
||||
&& mAppsView.getSearchUiManager().getEditText() != null) {
|
||||
mAppsView.getSearchUiManager().getEditText().requestFocus();
|
||||
}
|
||||
if (toAllApps) {
|
||||
InteractionJankMonitorWrapper.end(Cuj.CUJ_LAUNCHER_OPEN_ALL_APPS);
|
||||
}
|
||||
}
|
||||
|
||||
/** Invoked on back press, returning {@code true} if the search session handled it. */
|
||||
|
||||
Reference in New Issue
Block a user