Always cancel forwarding on ACTION_UP or ACTION_CANCEL.

Bug: 30563284
Change-Id: I6285eda6d9f7a775f8ae896baebc420a0cf84e21
This commit is contained in:
Tony Wickham
2016-08-01 13:06:40 -07:00
parent 0843675e93
commit 37a759a99c
@@ -218,8 +218,12 @@ public class ShortcutsContainerListener implements View.OnTouchListener,
if (dst == null) {
return false;
}
// Always cancel forwarding when the touch stream ends.
final int action = srcEvent.getActionMasked();
final boolean keepForwarding = action != MotionEvent.ACTION_UP
&& action != MotionEvent.ACTION_CANCEL;
if (!dst.isLaidOut()) {
return true;
return keepForwarding;
}
// Convert event to destination-local coordinates.
@@ -236,11 +240,6 @@ public class ShortcutsContainerListener implements View.OnTouchListener,
final boolean handled = dst.onForwardedEvent(dstEvent, mActivePointerId, mTouchDown);
dstEvent.recycle();
// Always cancel forwarding when the touch stream ends.
final int action = srcEvent.getActionMasked();
final boolean keepForwarding = action != MotionEvent.ACTION_UP
&& action != MotionEvent.ACTION_CANCEL;
return handled && keepForwarding;
}