-
bdakin authored
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: http://svn.webkit.org/repository/webkit/trunk@18057 268f45cc-cd09-0410-ab3c-d52691b4dbfc
38765cde