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

content/browser/renderer_host/render_widget_host_unittest.cc

Index: content/browser/renderer_host/render_widget_host_unittest.cc
diff --git a/content/browser/renderer_host/render_widget_host_unittest.cc b/content/browser/renderer_host/render_widget_host_unittest.cc
index 69a7bcf618efd46c41a916473341b90d5ef10e65..0a952c041ea5a479cff2d862a5f00ccbfc7df5bc 100644
--- a/content/browser/renderer_host/render_widget_host_unittest.cc
+++ b/content/browser/renderer_host/render_widget_host_unittest.cc
@@ -32,6 +32,7 @@
 #include "content/test/test_render_view_host.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "ui/display/screen.h"
+#include "ui/events/base_event_utils.h"
 #include "ui/events/blink/web_input_event_traits.h"
 #include "ui/events/keycodes/keyboard_codes.h"
 #include "ui/gfx/canvas.h"
@@ -259,7 +260,7 @@ class TestView : public TestRenderWidgetHostView {
   const WebTouchEvent& acked_event() const { return acked_event_; }
   int acked_event_count() const { return acked_event_count_; }
   void ClearAckedEvent() {
-    acked_event_.type = blink::WebInputEvent::Undefined;
+    acked_event_.setType(blink::WebInputEvent::Undefined);
     acked_event_count_ = 0;
   }
 
@@ -311,8 +312,8 @@ class TestView : public TestRenderWidgetHostView {
     // into WebInputEvent::MouseMove.)
     WebMouseEvent event =
         SyntheticWebMouseEventBuilder::Build(WebInputEvent::MouseMove);
-    event.timeStampSeconds =
-        (base::TimeTicks::Now() - base::TimeTicks()).InSecondsF();
+    event.setTimeStampSeconds(
+        ui::EventTimeStampToSeconds(ui::EventTimeForNow()));
     rwh_->input_router()->SendMouseEvent(
         MouseEventWithLatencyInfo(event, ui::LatencyInfo()));
   }
@@ -448,7 +449,7 @@ class RenderWidgetHostTest : public testing::Test {
         handle_mouse_event_(false),
         simulated_event_time_delta_seconds_(0) {
     last_simulated_event_time_seconds_ =
-        (base::TimeTicks::Now() - base::TimeTicks()).InSecondsF();
+        ui::EventTimeStampToSeconds(ui::EventTimeForNow());
   }
   ~RenderWidgetHostTest() override {}
 
@@ -545,10 +546,7 @@ class RenderWidgetHostTest : public testing::Test {
   }
 
   void SimulateKeyboardEvent(WebInputEvent::Type type, int modifiers) {
-    WebKeyboardEvent event = SyntheticWebKeyboardEventBuilder::Build(type);
-    event.modifiers = modifiers;
-    NativeWebKeyboardEvent native_event;
-    memcpy(&native_event, &event, sizeof(event));
+    NativeWebKeyboardEvent native_event(type, modifiers, ui::EventTimeForNow());

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

Shouldn't this be the simulate time i.e., GetNextSimulatedEventTimeSeconds?

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

Done.

     host_->ForwardKeyboardEvent(native_event);
   }
 
@@ -589,7 +587,7 @@ class RenderWidgetHostTest : public testing::Test {
         SyntheticWebMouseEventBuilder::Build(type, x, y, modifiers);
     if (pressed)
       event.button = WebMouseEvent::Button::Left;
-    event.timeStampSeconds = GetNextSimulatedEventTimeSeconds();
+    event.setTimeStampSeconds(GetNextSimulatedEventTimeSeconds());
     host_->ForwardMouseEvent(event);
   }