Fix server-side image map click location with "Open link in new tab"

third_party/WebKit/Source/core/page/ContextMenuController.cpp

Index: third_party/WebKit/Source/core/page/ContextMenuController.cpp
diff --git a/third_party/WebKit/Source/core/page/ContextMenuController.cpp b/third_party/WebKit/Source/core/page/ContextMenuController.cpp
index 515a3e18c942d861d8dcb83dd00d236a05e8ae49..2de007b7b6b8db53393e1193414a963bc2ed36ff 100644
--- a/third_party/WebKit/Source/core/page/ContextMenuController.cpp
+++ b/third_party/WebKit/Source/core/page/ContextMenuController.cpp
@@ -173,7 +173,7 @@ PassOwnPtr<ContextMenu> ContextMenuController::createContextMenu(LocalFrame* fra
 
 void ContextMenuController::showContextMenu(Event* event)
 {
-    m_client->showContextMenu(m_contextMenu.get());
+    m_client->showContextMenu(m_contextMenu.get(), event);

[tkent, 2016/02/05 02:05:47]

|event| can be nullptr.  It causes a crash in appendServerMapMousePosition().

[JakeYoon, 2016/02/05 04:04:49]

How do you think to add to check nullptr in appendServerMapMousePosition()
function?

[zino, 2016/02/05 05:47:34]

I'm not owner but I think you could modify appendServerMapMousePosition() so
that the method takes mouse position instead of event object.
Then, you can get the current position from hit test result in
ContextMenuClientImpl::showContextMenu() without passing event object.

[tkent, 2016/02/05 06:00:45]

Check it in ContextMenuClientImpl::showContextMenu is cleaner, I think.

[JakeYoon, 2016/02/05 07:28:34]

Okay, I think so too

     if (event)
         event->setDefaultHandled();
 }