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

Unified Diff: ui/base/gestures/gesture_sequence.cc

Issue 10702096: Corrected issues with queued touch events. This fixes gestures in Windows. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Fix for multi-touch gestures. We now track the number of touch events in each message, and send th… Created 8 years, 5 months 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: ui/base/gestures/gesture_sequence.cc
diff --git a/ui/base/gestures/gesture_sequence.cc b/ui/base/gestures/gesture_sequence.cc
index 9892fdce2451038d0c28c843777b5bf695fdaac9..c766e76dbb88bdfcf4a445b81e90f7ed5daff09b 100644
--- a/ui/base/gestures/gesture_sequence.cc
+++ b/ui/base/gestures/gesture_sequence.cc
@@ -236,6 +236,7 @@ GestureSequence::Gestures* GestureSequence::ProcessTouchEventForGesture(
// We shouldn't be able to get two PRESSED events from the same
// finger without either a RELEASE or CANCEL in between.
DCHECK(!points_[event.GetTouchId()].in_use());
+ DCHECK(!new_point->in_use());
sadrul 2012/07/05 17:00:54 lines 238/239 are checking the same thing?
girard 2012/07/05 18:43:44 Yes. Since the rest of the code uses the new_poin
sadrul 2012/07/09 07:12:06 Ah, in that case, remove line 238?
girard 2012/07/26 12:36:43 Done.
new_point->set_point_id(point_count_++);
new_point->set_touch_id(event.GetTouchId());
}

Powered by Google App Engine
This is Rietveld 408576698