Remove PlatformMouseEvent and use WebMouseEvent instead

third_party/WebKit/Source/web/PopupMenuImpl.cpp

Index: third_party/WebKit/Source/web/PopupMenuImpl.cpp
diff --git a/third_party/WebKit/Source/web/PopupMenuImpl.cpp b/third_party/WebKit/Source/web/PopupMenuImpl.cpp
index 7b2b88f4b059ea43c4ecfec2b2b5485d281c1ffc..97bf482a6c9ff11ee298c762097a9b012dd0367e 100644
--- a/third_party/WebKit/Source/web/PopupMenuImpl.cpp
+++ b/third_party/WebKit/Source/web/PopupMenuImpl.cpp
@@ -446,7 +446,9 @@ void PopupMenuImpl::setValueAndClosePopup(int numValue,
   // We dispatch events on the owner element to match the legacy behavior.
   // Other browsers dispatch click events before and after showing the popup.
   if (m_ownerElement) {
-    PlatformMouseEvent event;
+    // TODO(dtapuska): Why is this event positionless?

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

It looks like this is called from popup HTML content, e.g. when a DateTime
picker selects an item using the enter key. It seems like we dispatch these
mouse events for compatibility but they don't actually come from a mouse click
so there's no position to set. I suppose we could pass the "last known mouse
position" but that doesn't seem easily accessible from here.

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

This should probably be generated from the element's position no? kind of like
how the context menu is generated...

[bokan, 2017/01/27 21:08:47]

Hmm, yeah, that makes sense.

+    WebMouseEvent event;
+    event.setFrameScale(1);
     Element* owner = &ownerElement();
     owner->dispatchMouseEvent(event, EventTypeNames::mouseup);
     owner->dispatchMouseEvent(event, EventTypeNames::click);