diff options
| author | varunjain@chromium.org <varunjain@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-06-07 15:33:05 +0000 |
|---|---|---|
| committer | varunjain@chromium.org <varunjain@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-06-07 15:33:05 +0000 |
| commit | f884a00aa8fe68434d1fe20e4b950bec30a4d2cc (patch) | |
| tree | d4b494c5539709a26b19369f96321d9f726a0902 /ash/wm | |
| parent | 1fa6de0736dc622711356cde6c08b8f95aa597ce (diff) | |
| download | chromium_src-f884a00aa8fe68434d1fe20e4b950bec30a4d2cc.zip chromium_src-f884a00aa8fe68434d1fe20e4b950bec30a4d2cc.tar.gz chromium_src-f884a00aa8fe68434d1fe20e4b950bec30a4d2cc.tar.bz2 | |
Fix TAP_DOWN gesture event:
1. GESTURE_TAP_DOWN should be sent only on the first TOUCH_PRESSED.
2. New gesture GESTURE_BEGIN shuold be sent on all TOUCH_PRESSED.
3. Rename GESTURE_TAP_UP to GESTURE_END.
BUG=131007
TEST=manual
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=140967
Review URL: https://chromiumcodereview.appspot.com/10533029
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@141011 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'ash/wm')
| -rw-r--r-- | ash/wm/system_gesture_event_filter.cc | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/ash/wm/system_gesture_event_filter.cc b/ash/wm/system_gesture_event_filter.cc index 99945ea..e7800fee 100644 --- a/ash/wm/system_gesture_event_filter.cc +++ b/ash/wm/system_gesture_event_filter.cc @@ -260,7 +260,7 @@ class SystemPinchHandler { return SYSTEM_GESTURE_END; switch (event.type()) { - case ui::ET_GESTURE_TAP_UP: { + case ui::ET_GESTURE_END: { if (event.delta_x() > kSystemPinchPoints) break; @@ -535,7 +535,7 @@ ui::GestureStatus SystemGestureEventFilter::PreHandleGestureEvent( ClearGestureHandlerForWindow(system_target); return ui::GESTURE_STATUS_CONSUMED; } else { - if (event->type() == ui::ET_GESTURE_TAP_DOWN && + if (event->type() == ui::ET_GESTURE_BEGIN && event->delta_x() >= kSystemPinchPoints) { pinch_handlers_[system_target] = new SystemPinchHandler(system_target); system_target->AddObserver(this); |