Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(36)

Unified Diff: content/public/android/java/src/org/chromium/content/browser/ContentViewGestureHandler.java

Issue 78263003: Ignore TouchCancel ack dispositions (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Speculative fix... Created 7 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/public/android/java/src/org/chromium/content/browser/ContentViewGestureHandler.java
diff --git a/content/public/android/java/src/org/chromium/content/browser/ContentViewGestureHandler.java b/content/public/android/java/src/org/chromium/content/browser/ContentViewGestureHandler.java
index 725667153c1f1684eed203533de3fbefb7591258..1f2ad9b688bb459f9d13748c0cc9875a0dd68065 100644
--- a/content/public/android/java/src/org/chromium/content/browser/ContentViewGestureHandler.java
+++ b/content/public/android/java/src/org/chromium/content/browser/ContentViewGestureHandler.java
@@ -275,9 +275,10 @@ class ContentViewGestureHandler implements LongPressDelegate {
// The ACK to the original event is received after timeout.
// Inject a touchcancel event.
mPendingAckState = PENDING_ACK_CANCEL_EVENT;
- mMotionEventDelegate.sendTouchEvent(mEventTime + TOUCH_EVENT_TIMEOUT,
- TouchPoint.TOUCH_EVENT_TYPE_CANCEL, mTouchPoints);
+ final TouchPoint[] touchPoints = mTouchPoints;
mTouchPoints = null;
+ mMotionEventDelegate.sendTouchEvent(mEventTime + TOUCH_EVENT_TIMEOUT,
+ TouchPoint.TOUCH_EVENT_TYPE_CANCEL, touchPoints);
return true;
case PENDING_ACK_CANCEL_EVENT:
TraceEvent.instant("TouchEventTimeout:ConfirmCancelEvent");

Powered by Google App Engine
This is Rietveld 408576698