Add constructors to WebInputEvents and setters so we can work at cleaning up these public structs.

blimp/net/input_message_unittest.cc

Index: blimp/net/input_message_unittest.cc
diff --git a/blimp/net/input_message_unittest.cc b/blimp/net/input_message_unittest.cc
index 1c2fc434c096a63cb5c836e2dac86ad3d5bef11b..0c2fda412756c1529efee516f0bf6fa2941fa60d 100644
--- a/blimp/net/input_message_unittest.cc
+++ b/blimp/net/input_message_unittest.cc
@@ -34,9 +34,8 @@ void ValidateWebGestureEventRoundTripping(const blink::WebGestureEvent& event) {
   EXPECT_EQ(0, memcmp(&event, new_event.get(), event.size));
 }
 
-blink::WebGestureEvent BuildBaseTestEvent() {
-  blink::WebGestureEvent event;
-  event.timeStampSeconds = 1.23;
+blink::WebGestureEvent BuildBaseTestEvent(blink::WebInputEvent::Type type) {
+  blink::WebGestureEvent event(type, blink::WebInputEvent::NoModifiers, 1.23);

[majidvp, 2016/12/19 20:09:52]

nit: s/1.23/TimeStampForTesting/

[dtapuska, 2016/12/20 19:49:21]

done; but I was trying to keep the original code but I agree this is better.

   event.x = 2;
   event.y = 3;
   event.globalX = 4;
@@ -48,8 +47,8 @@ blink::WebGestureEvent BuildBaseTestEvent() {
 }  // namespace
 
 TEST(InputMessageTest, TestGestureScrollBeginRoundTrip) {
-  blink::WebGestureEvent event = BuildBaseTestEvent();
-  event.type = blink::WebInputEvent::Type::GestureScrollBegin;
+  blink::WebGestureEvent event =
+      BuildBaseTestEvent(blink::WebInputEvent::Type::GestureScrollBegin);
   event.data.scrollBegin.deltaXHint = 2.34f;
   event.data.scrollBegin.deltaYHint = 3.45f;
   event.data.scrollBegin.targetViewport = true;
@@ -57,14 +56,14 @@ TEST(InputMessageTest, TestGestureScrollBeginRoundTrip) {
 }
 
 TEST(InputMessageTest, TestGestureScrollEndRoundTrip) {
-  blink::WebGestureEvent event = BuildBaseTestEvent();
-  event.type = blink::WebInputEvent::Type::GestureScrollEnd;
+  blink::WebGestureEvent event =
+      BuildBaseTestEvent(blink::WebInputEvent::Type::GestureScrollEnd);
   ValidateWebGestureEventRoundTripping(event);
 }
 
 TEST(InputMessageTest, TestGestureScrollUpdateRoundTrip) {
-  blink::WebGestureEvent event = BuildBaseTestEvent();
-  event.type = blink::WebInputEvent::Type::GestureScrollUpdate;
+  blink::WebGestureEvent event =
+      BuildBaseTestEvent(blink::WebInputEvent::Type::GestureScrollUpdate);
   event.data.scrollUpdate.deltaX = 2.34f;
   event.data.scrollUpdate.deltaY = 3.45f;
   event.data.scrollUpdate.velocityX = 4.56f;
@@ -76,8 +75,8 @@ TEST(InputMessageTest, TestGestureScrollUpdateRoundTrip) {
 }
 
 TEST(InputMessageTest, TestGestureFlingStartRoundTrip) {
-  blink::WebGestureEvent event = BuildBaseTestEvent();
-  event.type = blink::WebInputEvent::Type::GestureFlingStart;
+  blink::WebGestureEvent event =
+      BuildBaseTestEvent(blink::WebInputEvent::Type::GestureFlingStart);
   event.data.flingStart.velocityX = 2.34f;
   event.data.flingStart.velocityY = 3.45f;
   event.data.flingStart.targetViewport = true;
@@ -85,15 +84,15 @@ TEST(InputMessageTest, TestGestureFlingStartRoundTrip) {
 }
 
 TEST(InputMessageTest, TestGestureFlingCancelRoundTrip) {
-  blink::WebGestureEvent event = BuildBaseTestEvent();
-  event.type = blink::WebInputEvent::Type::GestureFlingCancel;
+  blink::WebGestureEvent event =
+      BuildBaseTestEvent(blink::WebInputEvent::Type::GestureFlingCancel);
   event.data.flingCancel.preventBoosting = true;
   ValidateWebGestureEventRoundTripping(event);
 }
 
 TEST(InputMessageTest, TestGestureTapRoundTrip) {
-  blink::WebGestureEvent event = BuildBaseTestEvent();
-  event.type = blink::WebGestureEvent::Type::GestureTap;
+  blink::WebGestureEvent event =
+      BuildBaseTestEvent(blink::WebGestureEvent::Type::GestureTap);
   event.data.tap.tapCount = 3;
   event.data.tap.width = 2.34f;
   event.data.tap.height = 3.45f;
@@ -101,20 +100,20 @@ TEST(InputMessageTest, TestGestureTapRoundTrip) {
 }
 
 TEST(InputMessageTest, TestGesturePinchBeginRoundTrip) {
-  blink::WebGestureEvent event = BuildBaseTestEvent();
-  event.type = blink::WebInputEvent::Type::GesturePinchBegin;
+  blink::WebGestureEvent event =
+      BuildBaseTestEvent(blink::WebInputEvent::Type::GesturePinchBegin);
   ValidateWebGestureEventRoundTripping(event);
 }
 
 TEST(InputMessageTest, TestGesturePinchEndRoundTrip) {
-  blink::WebGestureEvent event = BuildBaseTestEvent();
-  event.type = blink::WebInputEvent::Type::GesturePinchEnd;
+  blink::WebGestureEvent event =
+      BuildBaseTestEvent(blink::WebInputEvent::Type::GesturePinchEnd);
   ValidateWebGestureEventRoundTripping(event);
 }
 
 TEST(InputMessageTest, TestGesturePinchUpdateRoundTrip) {
-  blink::WebGestureEvent event = BuildBaseTestEvent();
-  event.type = blink::WebInputEvent::Type::GesturePinchUpdate;
+  blink::WebGestureEvent event =
+      BuildBaseTestEvent(blink::WebInputEvent::Type::GesturePinchUpdate);
   event.data.pinchUpdate.zoomDisabled = true;
   event.data.pinchUpdate.scale = 2.34f;
   ValidateWebGestureEventRoundTripping(event);
@@ -123,8 +122,9 @@ TEST(InputMessageTest, TestGesturePinchUpdateRoundTrip) {
 TEST(InputMessageTest, TestUnsupportedInputEventSerializationFails) {
   // We currently do not support MouseWheel events.  If that changes update
   // this test.
-  blink::WebGestureEvent event;
-  event.type = blink::WebInputEvent::Type::MouseWheel;
+  blink::WebGestureEvent event(blink::WebInputEvent::Type::MouseWheel,
+                               blink::WebInputEvent::NoModifiers,
+                               blink::WebInputEvent::TimeStampForTesting);
   InputMessageGenerator generator;
   EXPECT_EQ(nullptr, generator.GenerateMessage(event).get());
 }
@@ -139,22 +139,22 @@ TEST(InputMessageConverterTest, TestTextInputTypeToProtoConversion) {
 }
 
 TEST(InputMessageTest, TestGestureTapDownRoundTrip) {
-  blink::WebGestureEvent event = BuildBaseTestEvent();
-  event.type = blink::WebGestureEvent::Type::GestureTapDown;
+  blink::WebGestureEvent event =
+      BuildBaseTestEvent(blink::WebGestureEvent::Type::GestureTapDown);
   event.data.tapDown.width = 2.3f;
   event.data.tapDown.height = 3.4f;
   ValidateWebGestureEventRoundTripping(event);
 }
 
 TEST(InputMessageTest, TestGestureTapCancelRoundTrip) {
-  blink::WebGestureEvent event = BuildBaseTestEvent();
-  event.type = blink::WebGestureEvent::Type::GestureTapCancel;
+  blink::WebGestureEvent event =
+      BuildBaseTestEvent(blink::WebGestureEvent::Type::GestureTapCancel);
   ValidateWebGestureEventRoundTripping(event);
 }
 
 TEST(InputMessageTest, TestGestureTapUnconfirmedRoundTrip) {
-  blink::WebGestureEvent event = BuildBaseTestEvent();
-  event.type = blink::WebGestureEvent::Type::GestureTapUnconfirmed;
+  blink::WebGestureEvent event =
+      BuildBaseTestEvent(blink::WebGestureEvent::Type::GestureTapUnconfirmed);
   event.data.tap.tapCount = 2;
   event.data.tap.width = 2.3f;
   event.data.tap.height = 3.4f;
@@ -162,8 +162,8 @@ TEST(InputMessageTest, TestGestureTapUnconfirmedRoundTrip) {
 }
 
 TEST(InputMessageTest, TestGestureShowPressRoundTrip) {
-  blink::WebGestureEvent event = BuildBaseTestEvent();
-  event.type = blink::WebGestureEvent::Type::GestureShowPress;
+  blink::WebGestureEvent event =
+      BuildBaseTestEvent(blink::WebGestureEvent::Type::GestureShowPress);
   event.data.showPress.width = 2.3f;
   event.data.showPress.height = 3.4f;
   ValidateWebGestureEventRoundTripping(event);