WebCore:
authorbdakin <bdakin@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 7 Dec 2006 19:49:21 +0000 (19:49 +0000)
committerbdakin <bdakin@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 7 Dec 2006 19:49:21 +0000 (19:49 +0000)
commitba3e325df336a61712feb8de9c396e4b874eb4aa
treedbeb5b1d250a0b8136b30107985442239231d576
parentd2806db2ae7af28ce17e25fe2e582a364b51e161
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
WebCore/ChangeLog
WebCore/page/ContextMenuClient.h
WebCore/page/ContextMenuController.cpp
WebKit/ChangeLog
WebKit/WebCoreSupport/WebContextMenuClient.h
WebKit/WebCoreSupport/WebContextMenuClient.mm
WebKit/WebKit.xcodeproj/project.pbxproj