Bundle DidOverscrollParams with the InputEventAck

content/common/input/input_event_ack.h

Index: content/common/input/input_event_ack.h
diff --git a/content/common/input/input_event_ack.h b/content/common/input/input_event_ack.h
new file mode 100644
index 0000000000000000000000000000000000000000..32ba6801df6fb37ac472f5db29fe9fa396a961cf
--- /dev/null
+++ b/content/common/input/input_event_ack.h
@@ -0,0 +1,29 @@
+// Copyright 2014 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_COMMON_INPUT_INPUT_EVENT_ACK_H_
+#define CONTENT_COMMON_INPUT_INPUT_EVENT_ACK_H_
+
+#include "base/memory/scoped_ptr.h"
+#include "content/common/content_export.h"
+#include "content/common/input/did_overscroll_params.h"
+#include "content/common/input/input_event_ack_state.h"
+#include "third_party/WebKit/public/web/WebInputEvent.h"
+#include "ui/events/latency_info.h"
+
+namespace content {
+
+struct CONTENT_EXPORT InputEventAck {

[piman, 2014/05/19 21:51:13]

nit: do you need this as an explicit separate struct, or would defining it in
input_messages.h as an IPC_STRUCT_BEGIN/END (instead of
IPC_STRUCT_TRAITS_BEGIN/END) be enough? The former is a little better in that
you don't have to worry about keeping the _messages.h file in sync with this
one.

[jdduke (slow), 2014/05/19 22:00:21]

Not particularly, defining in input_messages.h works just as well.

+  InputEventAck();
+  ~InputEventAck();
+
+  blink::WebInputEvent::Type type;
+  InputEventAckState state;
+  ui::LatencyInfo latency;
+  scoped_ptr<DidOverscrollParams> overscroll;
+};
+
+}  // namespace content
+
+#endif  // CONTENT_COMMON_INPUT_INPUT_EVENT_ACK_H_