Events ignoring ack disposition receive synthetic acks.

content/browser/renderer_host/input/gesture_event_filter.h

Index: content/browser/renderer_host/input/gesture_event_filter.h
diff --git a/content/browser/renderer_host/input/gesture_event_filter.h b/content/browser/renderer_host/input/gesture_event_filter.h
index 7ed57decd304289eb2127459ebf789b756e71ecd..f46f0f3fd69aed720f9a30b9ca42085d9d877118 100644
--- a/content/browser/renderer_host/input/gesture_event_filter.h
+++ b/content/browser/renderer_host/input/gesture_event_filter.h
@@ -92,8 +92,7 @@ class CONTENT_EXPORT GestureEventFilter {
  private:
   friend class MockRenderWidgetHost;
   friend class GestureEventFilterTest;
-
-  static bool ShouldIgnoreAckForGestureType(blink::WebInputEvent::Type type);
+  friend class ImmediateInputRouterTest;

[jdduke (slow), 2013/11/07 23:12:52]

I think we can test sync ack behavior in the ImmediateInputRouterTest without
needing to know the internals of the GEF.  Just make sure we get the right
number of sent events and acked events.

[tdresser, 2013/11/08 16:20:34]

Done.

 
   // TODO(mohsen): There are a bunch of ShouldForward.../ShouldDiscard...
   // methods that are getting confusing. This should be somehow fixed. Maybe
@@ -152,11 +151,6 @@ class CONTENT_EXPORT GestureEventFilter {
   gfx::Transform GetTransformForEvent(
       const GestureEventWithLatencyInfo& gesture_event) const;
 
-  // Pops and sends events ignoring ack from the head of
-  // |coalesced_gesture_events_| until the queue is empty or the event at the
-  // head requires an ack.
-  void SendEventsIgnoringAck();
-
   // Adds |gesture_event| to the |coalesced_gesture_events_|, resetting the
   // accumulation of |combined_scroll_pinch_|.
   void EnqueueEvent(const GestureEventWithLatencyInfo& gesture_event);