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

third_party/WebKit/Source/web/ContextMenuClientImpl.cpp

 /*
  * Copyright (C) 2009, 2012 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 113 matching lines @@
     RefPtrWillBeRawPtr<Range> markerRange = selectionRange->cloneRange();
     markerRange->setStart(markerRange->startContainer(), markers[0]->startOffset());
     markerRange->setEnd(markerRange->endContainer(), markers[0]->endOffset());
 
     if (markerRange->text().stripWhiteSpace(&IsWhiteSpaceOrPunctuation) != selectionRange->text().stripWhiteSpace(&IsWhiteSpaceOrPunctuation))
         return String();
 
     return markerRange->text();
 }
 
-void ContextMenuClientImpl::showContextMenu(const ContextMenu* defaultMenu)
+void ContextMenuClientImpl::showContextMenu(const ContextMenu* defaultMenu, Event* event)
 {
     // Displaying the context menu in this function is a big hack as we don't
     // have context, i.e. whether this is being invoked via a script or in
     // response to user input (Mouse event WM_RBUTTONDOWN,
     // Keyboard events KeyVK_APPS, Shift+F10). Check if this is being invoked
     // in response to the above input events before popping up the context menu.
     if (!ContextMenuAllowedScope::isContextMenuAllowed())
         return;
 
     HitTestResult r = m_webView->page()->contextMenuController().hitTestResult();
@@ skipping 18 matching lines @@
     if (toLocalFrame(m_webView->focusedCoreFrame())->editor().canPaste())
         data.editFlags |= WebContextMenuData::CanPaste;
     if (toLocalFrame(m_webView->focusedCoreFrame())->editor().canDelete())
         data.editFlags |= WebContextMenuData::CanDelete;
     // We can always select all...
     data.editFlags |= WebContextMenuData::CanSelectAll;
     data.editFlags |= WebContextMenuData::CanTranslate;
 
     // Links, Images, Media tags, and Image/Media-Links take preference over
     // all else.
-    data.linkURL = r.absoluteLinkURL();
+    StringBuilder url;

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

Even if the target node is not image map link, This showContextMenu() method
will always be called when contextmenu event happen.
But we don't need to run these codes at all times.

+    HTMLAnchorElement::appendServerMapMousePosition(url, event);
+    data.linkURL = KURL(r.absoluteLinkURL(), url.toString());
 
     if (r.innerNode()->isHTMLElement()) {
         HTMLElement* htmlElement = toHTMLElement(r.innerNode());
         if (!htmlElement->title().isEmpty()) {
             data.titleText = htmlElement->title();
         } else {
             data.titleText = htmlElement->altText();
         }
     }
 
@@ skipping 230 matching lines @@
         outputItems[i] = subItems[i];
     subMenuItems.swap(outputItems);
 }
 
 void ContextMenuClientImpl::populateCustomMenuItems(const ContextMenu* defaultMenu, WebContextMenuData* data)
 {
     populateSubMenuItems(defaultMenu->items(), data->customItems);
 }
 
 } // namespace blink