Mark touchcancel events as uncancelable

content/browser/renderer_host/input/touch_event_queue.cc

Index: content/browser/renderer_host/input/touch_event_queue.cc
diff --git a/content/browser/renderer_host/input/touch_event_queue.cc b/content/browser/renderer_host/input/touch_event_queue.cc
index 8662f4a293c8245bd13d60f79fbf0041b3850ad5..7bdae28c9ddf166708b56a8e4abc95408288b2a4 100644
--- a/content/browser/renderer_host/input/touch_event_queue.cc
+++ b/content/browser/renderer_host/input/touch_event_queue.cc
@@ -9,7 +9,7 @@
 #include "base/debug/trace_event.h"
 #include "base/stl_util.h"
 #include "content/browser/renderer_host/input/timeout_monitor.h"
-#include "content/browser/renderer_host/input/web_touch_event_traits.h"
+#include "content/common/input/web_touch_event_traits.h"
 #include "content/public/common/content_switches.h"
 #include "ui/gfx/geometry/point_f.h"
 
@@ -31,9 +31,11 @@ typedef std::vector<TouchEventWithLatencyInfo> WebTouchEventWithLatencyList;
 TouchEventWithLatencyInfo ObtainCancelEventForTouchEvent(
     const TouchEventWithLatencyInfo& event_to_cancel) {
   TouchEventWithLatencyInfo event = event_to_cancel;
-  event.event.type = WebInputEvent::TouchCancel;
-  for (size_t i = 0; i < event.event.touchesLength; i++)
-    event.event.touches[i].state = WebTouchPoint::StateCancelled;
+  WebTouchEventTraits::ResetType(&event.event,
+                                 WebInputEvent::TouchCancel,
+                                 // FIXME: Shouldn't we use a fresh timestamp?
+                                 event.event.timeStampSeconds,

[jdduke (slow), 2014/04/23 21:55:47]

Nah, I don't think it matters at this point.

[Rick Byers, 2014/04/23 23:15:35]

Why?  Shouldn't an app be able to see that a timeout touchcancel is always
generated ~150ms after the previous touch event?  Also I think apps may expect
touch events to arrive in monotonically increasing timeout value (eg. don't
expect a negative delta from one to the next). 

[jdduke (slow), 2014/04/23 23:29:49]

I think we're monotonically non-decreasing, and I see your point about the
timestamps.  Doesn't blink end up over-writing all of the provided timestamps
anyway? I guess that's still not a good reason to lie here... The FIXME is fine
:)

+                                 WebTouchEventTraits::RESET_TOUCHES_STATE);
   return event;
 }