Provided batched input delivery with a BufferedInputRouter

content/browser/renderer_host/input/event_ack_handler.h

+// Copyright (c) 2013 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.
+
+#ifndef CONTENT_BROWSER_RENDERER_HOST_INPUT_EVENT_ACK_HANDLER_H_
+#define CONTENT_BROWSER_RENDERER_HOST_INPUT_EVENT_ACK_HANDLER_H_
+
+#include "base/basictypes.h"
+
+namespace content {
+
+class EventInjector;
+class InputEvent;
+
+// Queued with an InputEvent to allow customized ack response.
+class EventAckHandler {

[aelias_OOO_until_Jul13, 2013/08/13 06:11:51]

Please delete this class as it stands, I don't think it makes sense in the
design.  More details below.

[jdduke (slow), 2013/08/13 15:29:48]

I agree, that's why I told you I wanted to delete it :)

+ public:
+  virtual ~EventAckHandler() { }
+
+  virtual void OnInputEventAck(const InputEvent& acked_event) = 0;

[aelias_OOO_until_Jul13, 2013/08/13 06:11:51]

This method can be moved to InputQueueClient.

[jdduke (slow), 2013/08/13 15:29:48]

Done.

+
+  // Allows the creation of follow-up events from an |acked_event|.
+  // Injected events will be inserted into the event queue at the same relative
+  // location as that occupied by |acked_event|.
+  class EventInjector {

[aelias_OOO_until_Jul13, 2013/08/13 06:11:51]

This class should be deleted (and it's not what I had in mind when I talked
about "EventInjector" in the design discussions).  If you need to hold your
place in the queue there's no need for a class like this for it.

[jdduke (slow), 2013/08/13 15:29:48]

Perhaps not in the end, but without modifying a number of external classes it
becomes a bit tricker (though by no means impossible).  As I said, I was always
in favor of deleting this...

+    public:
+    virtual ~EventInjector() { }
+    virtual void InjectEvent(const InputEvent& acked_event,
+                             EventAckHandler* ack_handler) = 0;
+  };
+
+  // Only called if |acked_event| was flagged for follow-up event injection.
+  // |injector| will never be null.
+  virtual void OnInputEventAck(const InputEvent& acked_event,
+                               EventInjector* injector) = 0;
+};
+
+} // namespace content
+
+#endif // CONTENT_BROWSER_RENDERER_HOST_INPUT_EVENT_ACK_HANDLER_H_