Suppressed MEs for gestures from cancelled PEs

ui/events/event.h

Index: ui/events/event.h
diff --git a/ui/events/event.h b/ui/events/event.h
index 114d73ad320cd7fdfe2753c397b0669f8c4a4f9e..57a683623dd6204ff415cb803effb5cc1cbee274 100644
--- a/ui/events/event.h
+++ b/ui/events/event.h
@@ -1055,6 +1055,13 @@ class EVENTS_EXPORT GestureEvent : public LocatedEvent {
                base::TimeDelta time_stamp,
                const GestureEventDetails& details);

[Rick Byers, 2016/06/08 20:42:48]

Should we plan to eliminate this ctor variant eventually?  Have you spot checked
the existing callers to see if any look like they could be sources of bugs (code
paths where the unique_touch_event_id gets dropped)?

[mustaq, 2016/06/08 20:53:16]

We have 83 calls to this ctor, only 4 of them are in non-test code:
src/ash/drag_drop/drag_drop_controller.cc
src/ui/events/gestures/gesture_provider_aura.cc
src/ui/events/ozone/evdev/event_factory_evdev.cc
src/ui/events/test/event_generator.cc

Only the third one is related to gestures released through touches which has
been fixed here.

Is there any way to mark the old ctor as test-only? Like Java
@VisibleForTesting?

 
+  GestureEvent(float x,
+               float y,
+               int flags,
+               base::TimeDelta time_stamp,
+               const GestureEventDetails& details,
+               uint32_t unique_touch_event_id);
+
   // Create a new GestureEvent which is identical to the provided model.
   // If source / target windows are provided, the model location will be
   // converted from |source| coordinate system to |target| coordinate system.
@@ -1068,6 +1075,10 @@ class EVENTS_EXPORT GestureEvent : public LocatedEvent {
 
   const GestureEventDetails& details() const { return details_; }
 
+  uint32_t unique_touch_event_id() const {
+    return unique_touch_event_id_;
+  }
+
  private:
   // For (de)serialization.
   GestureEvent(EventType type, base::TimeDelta time_stamp, int flags)
@@ -1076,6 +1087,7 @@ class EVENTS_EXPORT GestureEvent : public LocatedEvent {
   friend struct IPC::ParamTraits<ui::GestureEvent>;
 
   GestureEventDetails details_;
+  uint32_t unique_touch_event_id_;
 };
 
 }  // namespace ui