Corrected issues with queued touch events. This fixes gestures in Windows.

ui/base/gestures/gesture_sequence.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "ui/base/gestures/gesture_sequence.h"
 
 #include <cmath>
 
 #include "base/logging.h"
 #include "base/memory/scoped_ptr.h"
@@ skipping 218 matching lines @@
   if (event.GetTouchId() >= kMaxGesturePoints)
     return NULL;
 
   if (event.GetEventType() == ui::ET_TOUCH_PRESSED) {
     if (point_count_ == kMaxGesturePoints)
       return NULL;
     GesturePoint* new_point = &points_[event.GetTouchId()];
     // 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_point variable, I thought the
DCHECK should use it too.

[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());
   }
 
   GestureState last_state = state_;
 
   // NOTE: when modifying these state transitions, also update gestures.dot
   scoped_ptr<Gestures> gestures(new Gestures());
   GesturePoint& point = GesturePointForEvent(event);
   point.UpdateValues(event);
@@ skipping 675 matching lines @@
     return;
 
   // Since long press timer has been started, there should be a non-NULL point.
   const GesturePoint* point = GetPointByPointId(0);
   if (!ui::gestures::IsInsideManhattanSquare(point->first_touch_position(),
       event.GetLocation()))
     long_press_timer_->Stop();
 }
 
 }  // namespace ui