WebCore:
Reviewed by Brady.
Fixes "Open in new Window" item in WebCore ContextMenus. It wasn't
doing anything before, and now it works! Also some formatting
changes, etc.
* page/ContextMenuClient.h: Several of the parameters to several of
the functions here should be const or const references. And now
they are!
* page/ContextMenuController.cpp:
(WebCore::openNewWindow): Moved the open new window functionality
into a helper function since it is used by several menu item tags.
Now after calling chrome()->createWindow() we call chrome()->show()
on the new window, and so it appears!
(WebCore::ContextMenuController::contextMenuItemSelected): Call new
openNewWindow function to do the right thing. Shift some
formatting, use local variable for HitTestResult. Implement action
for ContextMenuItemTagOpenLink.
WebKit:
Reviewed by Brady.
Make some parameters const and const references.
* WebCoreSupport/WebContextMenuClient.h:
* WebCoreSupport/WebContextMenuClient.mm:
(WebContextMenuClient::contextMenuItemSelected):
(WebContextMenuClient::copyLinkToClipboard):
(WebContextMenuClient::downloadURL):
(WebContextMenuClient::copyImageToClipboard):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@18057
268f45cc-cd09-0410-ab3c-
d52691b4dbfc