WebMouseEvent coordinates are now fractional & private

third_party/WebKit/public/platform/WebMouseEvent.h

Index: third_party/WebKit/public/platform/WebMouseEvent.h
diff --git a/third_party/WebKit/public/platform/WebMouseEvent.h b/third_party/WebKit/public/platform/WebMouseEvent.h
index fee8a2572fe5cd5faf5746d128e2b097a15ad115..4602ade567425562c03636ebef821793c36126fc 100644
--- a/third_party/WebKit/public/platform/WebMouseEvent.h
+++ b/third_party/WebKit/public/platform/WebMouseEvent.h
@@ -21,12 +21,10 @@ class WebMouseEvent : public WebInputEvent, public WebPointerProperties {
   // Renderer coordinates. Similar to viewport coordinates but without
   // DevTools emulation transform or overscroll applied. i.e. the coordinates
   // in Chromium's RenderView bounds.
-  int x;
-  int y;
+  WebFloatPoint position;

[dtapuska, 2017/03/29 20:19:15]

I think we really should move these to be accessors like

positionInScreen()
and
positionInView()

that will go a long way in making this class struct like and more OO like.

[mustaq, 2017/03/30 21:01:35]

Done, all done, phew!

 
   // Screen coordinate
-  int globalX;
-  int globalY;
+  WebFloatPoint screenPosition;
 
   int clickCount;
 
@@ -42,10 +40,8 @@ class WebMouseEvent : public WebInputEvent, public WebPointerProperties {
                       modifiersParam,
                       timeStampSecondsParam),
         WebPointerProperties(),
-        x(xParam),
-        y(yParam),
-        globalX(globalXParam),
-        globalY(globalYParam) {}
+        position(xParam, yParam),
+        screenPosition(WebFloatPoint(globalXParam, globalYParam)) {}
 
   WebMouseEvent(Type typeParam,
                 WebFloatPoint position,
@@ -59,10 +55,8 @@ class WebMouseEvent : public WebInputEvent, public WebPointerProperties {
                       modifiersParam,
                       timeStampSecondsParam),
         WebPointerProperties(buttonParam, PointerType::Mouse),
-        x(position.x),
-        y(position.y),
-        globalX(globalPosition.x),
-        globalY(globalPosition.y),
+        position(position),
+        screenPosition(globalPosition),
         clickCount(clickCountParam) {}
 
   WebMouseEvent(Type typeParam,