Remove PlatformGestureEvent in favour of using WebGestureEvent

third_party/WebKit/Source/core/input/EventHandlerTest.cpp

Index: third_party/WebKit/Source/core/input/EventHandlerTest.cpp
diff --git a/third_party/WebKit/Source/core/input/EventHandlerTest.cpp b/third_party/WebKit/Source/core/input/EventHandlerTest.cpp
index fb48cc0a09e3c5090b279c0fd5f421499f84b904..71272117a1f21f4a3dfeced118ecdf7dc6adb5bf 100644
--- a/third_party/WebKit/Source/core/input/EventHandlerTest.cpp
+++ b/third_party/WebKit/Source/core/input/EventHandlerTest.cpp
@@ -33,17 +33,17 @@ class EventHandlerTest : public ::testing::Test {
   std::unique_ptr<DummyPageHolder> m_dummyPageHolder;
 };
 
-class TapEventBuilder : public PlatformGestureEvent {
+class TapEventBuilder : public WebGestureEvent {
  public:
-  TapEventBuilder(IntPoint position, int tapCount)
-      : PlatformGestureEvent(PlatformEvent::GestureTap,
-                             position,
-                             position,
-                             IntSize(5, 5),
-                             WTF::monotonicallyIncreasingTime(),
-                             static_cast<PlatformEvent::Modifiers>(0),
-                             PlatformGestureSourceTouchscreen) {
-    m_data.m_tap.m_tapCount = tapCount;
+  TapEventBuilder(IntPoint position, int tapCount) : WebGestureEvent() {
+    type = WebInputEvent::GestureTap;
+    x = globalX = position.x();
+    y = globalY = position.y();
+    timeStampSeconds = WTF::monotonicallyIncreasingTime();
+    sourceDevice = WebGestureDeviceTouchscreen;
+    data.tap.tapCount = tapCount;
+    data.tap.width = 5;
+    data.tap.height = 5;

[majidvp, 2016/12/01 15:46:30]

One of the advantage of PlatformGestureEvent was that it had
constructors which helped ensure we initialize the data members
correctly. 

At the moment, tt is possible to construct WebGestureEvent 
incorrectly e.g., very easy to forget to initialize timeStampSeconds. Is there a
way to avoid this?

[dtapuska, 2016/12/01 16:26:43]

So my eventual goal is to have WebInputEvent be a full C++ class and then we can
have constructors. That can be done independently. I don't care for the wild
west struct.

   }
 };