Remove PlatformMouseEvent and use WebMouseEvent instead

third_party/WebKit/Source/web/WebInputEventConversion.cpp

 /*
  * Copyright (C) 2009 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 67 matching lines @@
       overscrollOffset =
           rootView->page()->frameHost().chromeClient().elasticOverscroll();
     }
   }
   return FloatPoint(
       -offset.width() / scale + visualViewport.x() + overscrollOffset.width(),
       -offset.height() / scale + visualViewport.y() +
           overscrollOffset.height());
 }
 
-float scaleDeltaToWindow(const Widget* widget, float delta) {
-  return delta / frameScale(widget);
-}
-
-// This method converts from the renderer's coordinate space into Blink's root
-// frame coordinate space.  It's somewhat unique in that it takes into account
-// DevTools emulation, which applies a scale and offset in the root layer (see
-// updateRootLayerTransform in WebViewImpl) as well as the overscroll effect on
-// OSX.  This is in addition to the visual viewport "pinch-zoom" transformation
-// and is one of the few cases where the visual viewport is not equal to the
-// renderer's coordinate-space.
-FloatPoint convertHitPointToRootFrame(const Widget* widget,
-                                      FloatPoint pointInRendererViewport) {
-  float scale = 1;
-  IntSize offset;
-  IntPoint visualViewport;
-  FloatSize overscrollOffset;
-  if (widget) {
-    FrameView* rootView = toFrameView(widget->root());
-    if (rootView) {
-      scale = rootView->inputEventsScaleFactor();
-      offset = rootView->inputEventsOffsetForEmulation();
-      visualViewport = flooredIntPoint(rootView->page()
-                                           ->frameHost()
-                                           .visualViewport()
-                                           .visibleRect()
-                                           .location());
-      overscrollOffset =
-          rootView->page()->frameHost().chromeClient().elasticOverscroll();
-    }
-  }
-  return FloatPoint((pointInRendererViewport.x() - offset.width()) / scale +
-                        visualViewport.x() + overscrollOffset.width(),
-                    (pointInRendererViewport.y() - offset.height()) / scale +
-                        visualViewport.y() + overscrollOffset.height());
-}
-
-unsigned toPlatformModifierFrom(WebMouseEvent::Button button) {
-  if (button == WebMouseEvent::Button::NoButton)
-    return 0;
-
-  unsigned webMouseButtonToPlatformModifier[] = {
-      PlatformEvent::LeftButtonDown, PlatformEvent::MiddleButtonDown,
-      PlatformEvent::RightButtonDown};
-
-  return webMouseButtonToPlatformModifier[static_cast<int>(button)];
-}
-
 FloatPoint convertAbsoluteLocationForLayoutObjectFloat(
     const DoublePoint& location,
     const LayoutItem layoutItem) {
   return layoutItem.absoluteToLocal(FloatPoint(location), UseTransforms);
 }
 
 IntPoint convertAbsoluteLocationForLayoutObjectInt(
     const DoublePoint& location,
     const LayoutItem layoutItem) {
   return roundedIntPoint(
@@ skipping 19 matching lines @@
   webEvent.globalX = event.screenX();
   webEvent.globalY = event.screenY();
   webEvent.windowX = pointInRootFrame.x();
   webEvent.windowY = pointInRootFrame.y();
   IntPoint localPoint = convertAbsoluteLocationForLayoutObjectInt(
       event.absoluteLocation(), layoutItem);
   webEvent.x = localPoint.x();
   webEvent.y = localPoint.y();
 }
 
+unsigned toWebInputEventModifierFrom(WebMouseEvent::Button button) {
+  if (button == WebMouseEvent::Button::NoButton)
+    return 0;
+
+  unsigned webMouseButtonToPlatformModifier[] = {
+      WebInputEvent::LeftButtonDown, WebInputEvent::MiddleButtonDown,
+      WebInputEvent::RightButtonDown};
+
+  return webMouseButtonToPlatformModifier[static_cast<int>(button)];
+}
+
 }  // namespace
 
-// MakePlatformMouseEvent -----------------------------------------------------
+WebMouseEvent TransformWebMouseEvent(Widget* widget,
+                                     const WebMouseEvent& event) {
+  WebMouseEvent result = event;
 
-// TODO(mustaq): Add tests for this.
-PlatformMouseEventBuilder::PlatformMouseEventBuilder(Widget* widget,
-                                                     const WebMouseEvent& e) {
-  // FIXME: Widget is always toplevel, unless it's a popup. We may be able
-  // to get rid of this once we abstract popups into a WebKit API.
-  m_position = widget->convertFromRootFrame(
-      flooredIntPoint(convertHitPointToRootFrame(widget, IntPoint(e.x, e.y))));
-  m_globalPosition = IntPoint(e.globalX, e.globalY);
-  m_movementDelta = IntPoint(scaleDeltaToWindow(widget, e.movementX),
-                             scaleDeltaToWindow(widget, e.movementY));
-  m_modifiers = e.modifiers();
+  // TODO(dtapuska): Remove this translation. In the past blink has
+  // converted leaves into moves and not known about leaves. It should
+  // be educated about them.
+  if (event.type() == WebInputEvent::MouseEnter ||
+      event.type() == WebInputEvent::MouseLeave) {
+    result.setType(WebInputEvent::MouseMove);
+  }
 
-  m_timestamp = TimeTicks::FromSeconds(e.timeStampSeconds());
-  m_clickCount = e.clickCount;
-
-  m_pointerProperties = static_cast<WebPointerProperties>(e);
-
-  switch (e.type()) {
-    case WebInputEvent::MouseMove:
-    case WebInputEvent::MouseEnter:  // synthesize a move event
-    case WebInputEvent::MouseLeave:  // synthesize a move event
-      m_type = PlatformEvent::MouseMoved;
-      break;
-
-    case WebInputEvent::MouseDown:
-      m_type = PlatformEvent::MousePressed;
-      break;
-
-    case WebInputEvent::MouseUp:
-      m_type = PlatformEvent::MouseReleased;
-
-      // The MouseEvent spec requires that buttons indicates the state
-      // immediately after the event takes place. To ensure consistency
-      // between platforms here, we explicitly clear the button that is
-      // in the process of being released.
-      m_modifiers &= ~toPlatformModifierFrom(e.button);
-      break;
-
-    default:
-      NOTREACHED();
+  // TODO(dtapuska): Perhaps the event should be constructed correctly?

[bokan, 2017/01/27 16:57:17]

Please file bugs for these and put the bug # in the TODO.

[dtapuska, 2017/01/27 21:00:00]

Done.

+  if (event.type() == WebInputEvent::MouseUp) {
+    result.setModifiers(event.modifiers() &
+                        ~toWebInputEventModifierFrom(event.button));
   }
+  result.setFrameScale(frameScale(widget));
+  result.setFrameTranslate(frameTranslation(widget));
+  return result;
 }
 
 WebMouseWheelEvent TransformWebMouseWheelEvent(
     Widget* widget,
     const WebMouseWheelEvent& event) {
   WebMouseWheelEvent result = event;
   result.setFrameScale(frameScale(widget));
   result.setFrameTranslate(frameTranslation(widget));
   return result;
 }
@@ skipping 21 matching lines @@
 
 WebTouchEvent TransformWebTouchEvent(Widget* widget,
                                      const WebTouchEvent& event) {
   return TransformWebTouchEvent(frameScale(widget), frameTranslation(widget),
                                 event);
 }
 
 WebMouseEventBuilder::WebMouseEventBuilder(const Widget* widget,
                                            const LayoutItem layoutItem,
                                            const MouseEvent& event) {
+  if (event.nativeEvent()) {
+    // This is the slimplified code path once we OOPIF ships

[bokan, 2017/01/27 16:57:17]

nit: "we OOPIF" -> "OOPIF"

I think this comment should be reworded - I'm not sure what it's trying to say.
Do we have to fix this once OOPIF ships or is this a hack to keep OOPIF working?

[dtapuska, 2017/01/27 21:00:00]

Done.

+    // as that will break sending synthetic events to OOPIFs and
+    // all events should have a native event attached.
+    *static_cast<WebMouseEvent*>(this) =
+        event.nativeEvent()->flattenTransform();
+    WebFloatPoint absoluteRootFrameLocation =
+        event.nativeEvent()->positionInRootFrame();
+    IntPoint localPoint = roundedIntPoint(
+        layoutItem.absoluteToLocal(absoluteRootFrameLocation, UseTransforms));
+    x = localPoint.x();
+    y = localPoint.y();
+    return;
+  }
+
   if (event.type() == EventTypeNames::mousemove)
     m_type = WebInputEvent::MouseMove;
   else if (event.type() == EventTypeNames::mouseout)
     m_type = WebInputEvent::MouseLeave;
   else if (event.type() == EventTypeNames::mouseover)
     m_type = WebInputEvent::MouseEnter;
   else if (event.type() == EventTypeNames::mousedown)
     m_type = WebInputEvent::MouseDown;
   else if (event.type() == EventTypeNames::mouseup)
     m_type = WebInputEvent::MouseUp;
@@ skipping 30 matching lines @@
         break;
     }
   } else {
     button = WebMouseEvent::Button::NoButton;
   }
   movementX = event.movementX();
   movementY = event.movementY();
   clickCount = event.detail();
 
   pointerType = WebPointerProperties::PointerType::Mouse;
-  if (event.mouseEvent())
-    pointerType = event.mouseEvent()->pointerProperties().pointerType;
 }
 
 // Generate a synthetic WebMouseEvent given a TouchEvent (eg. for emulating a
 // mouse with touch input for plugins that don't support touch input).
 WebMouseEventBuilder::WebMouseEventBuilder(const Widget* widget,
                                            const LayoutItem layoutItem,
                                            const TouchEvent& event) {
   if (!event.touches())
     return;
   if (event.touches()->length() != 1) {
@@ skipping 66 matching lines @@
   else if (event.type() == EventTypeNames::keypress)
     m_type = WebInputEvent::Char;
   else
     return;  // Skip all other keyboard events.
 
   m_modifiers = event.modifiers();
   m_timeStampSeconds = event.platformTimeStamp().InSeconds();
   windowsKeyCode = event.keyCode();
 }
 
-Vector<PlatformMouseEvent> createPlatformMouseEventVector(
+Vector<WebMouseEvent> TransformWebMouseEventVector(
     Widget* widget,
     const std::vector<const WebInputEvent*>& coalescedEvents) {
-  Vector<PlatformMouseEvent> result;
+  Vector<WebMouseEvent> result;
   for (const auto& event : coalescedEvents) {
     DCHECK(WebInputEvent::isMouseEventType(event->type()));
-    result.push_back(PlatformMouseEventBuilder(
+    result.push_back(TransformWebMouseEvent(
         widget, static_cast<const WebMouseEvent&>(*event)));
   }
   return result;
 }
 
 Vector<WebTouchEvent> TransformWebTouchEventVector(
     Widget* widget,
     const std::vector<const WebInputEvent*>& coalescedEvents) {
   float scale = frameScale(widget);
   FloatPoint translation = frameTranslation(widget);
   Vector<WebTouchEvent> result;
   for (const auto& event : coalescedEvents) {
     DCHECK(WebInputEvent::isTouchEventType(event->type()));
     result.push_back(TransformWebTouchEvent(
         scale, translation, static_cast<const WebTouchEvent&>(*event)));
   }
   return result;
 }
 
 }  // namespace blink