Commit a1567d50 authored Apr 15, 2020 by Jon Miranda

Prevent two overlapping long press actions.

We add a check in CheckLongPressHelper to not trigger long press if the
view is already in the pressed state.

This fixes the bug where a stranded DragView is left in the DragLayer.

Repo's every 50 tries:
- Open folder
- Long click and slightly drag the same item over and over in slightly
  different directions.

Logs show that long click on folder item initiates the first long click,
and then later on CheckLongPressHelper triggers another long press before the
initial one is finished.

Bug: 151053858
Change-Id: Ifbb6919e4f0153aaef2904f704baa284e78f1e83

parent a976a221

src/com/android/launcher3/CheckLongPressHelper.java

+4 −1

Original line number	Original line	Diff line number	Diff line
	@@ -113,7 +113,10 @@ public class CheckLongPressHelper {
	}		}

	private void triggerLongPress() {		private void triggerLongPress() {
	if ((mView.getParent() != null) && mView.hasWindowFocus() && !mHasPerformedLongPress) {		if ((mView.getParent() != null)
			&& mView.hasWindowFocus()
			&& (!mView.isPressed() \|\| mListener == null)
			&& !mHasPerformedLongPress) {
	boolean handled;		boolean handled;
	if (mListener != null) {		if (mListener != null) {
	handled = mListener.onLongClick(mView);		handled = mListener.onLongClick(mView);