Commit c3e55e1d authored by jdduke@chromium.org's avatar jdduke@chromium.org

[Android] Revert for regression where gestures interrupted on page load

This is a partial revert of https://codereview.chromium.org/120203003.
It is quite common for a page to generate multiple |DidStartLoading()|
notifications while it finishes loading.  Redundant notifications should
not reset a gesture mid-sequence.

BUG=335586

Review URL: https://codereview.chromium.org/141853002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@245630 0039d316-1c4b-4281-b951-d872f2087c98
parent 0a3a466a
...@@ -850,8 +850,6 @@ class ContentViewGestureHandler implements LongPressDelegate { ...@@ -850,8 +850,6 @@ class ContentViewGestureHandler implements LongPressDelegate {
mZoomManager.processTouchEvent(me); mZoomManager.processTouchEvent(me);
me.recycle(); me.recycle();
mLongPressDetector.cancelLongPress(); mLongPressDetector.cancelLongPress();
if (mCurrentDownEvent != null) recycleEvent(mCurrentDownEvent);
mCurrentDownEvent = null;
} }
/** /**
...@@ -935,6 +933,8 @@ class ContentViewGestureHandler implements LongPressDelegate { ...@@ -935,6 +933,8 @@ class ContentViewGestureHandler implements LongPressDelegate {
mTouchMoveToNativeConfirmed = false; mTouchMoveToNativeConfirmed = false;
mTouchDownToNativeX = event.getX(); mTouchDownToNativeX = event.getX();
mTouchDownToNativeY = event.getY(); mTouchDownToNativeY = event.getY();
if (mCurrentDownEvent != null) recycleEvent(mCurrentDownEvent);
mCurrentDownEvent = null;
} }
if (mTouchHandlingState == NO_TOUCH_HANDLER_FOR_GESTURE) return EVENT_NOT_FORWARDED; if (mTouchHandlingState == NO_TOUCH_HANDLER_FOR_GESTURE) return EVENT_NOT_FORWARDED;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment