• andersca@apple.com's avatar
    2011-05-13 Anders Carlsson <andersca@apple.com> · 4c8ba961
    andersca@apple.com authored
            Reviewed by Sam Weinig.
    
            NPN_GetURL with a javascript: URL doesn't send back the result in the stream
            https://bugs.webkit.org/show_bug.cgi?id=60810
    
            In NetscapePluginStream::sendJavaScriptStream, we were incorrectly passing the result URL
            to the stream, instead of the result. Also, store the request URL in the NetscapePluginStream object
            to avoid sending it over the wire.
    
            * PluginProcess/PluginControllerProxy.cpp:
            (WebKit::PluginControllerProxy::didEvaluateJavaScript):
            * PluginProcess/PluginControllerProxy.h:
            * PluginProcess/PluginControllerProxy.messages.in:
            * WebProcess/Plugins/Netscape/NetscapePlugin.cpp:
            (WebKit::NetscapePlugin::loadURL):
            (WebKit::NetscapePlugin::didEvaluateJavaScript):
            (WebKit::NetscapePlugin::manualStreamDidReceiveResponse):
            * WebProcess/Plugins/Netscape/NetscapePlugin.h:
            * WebProcess/Plugins/Netscape/NetscapePluginStream.cpp:
            (WebKit::NetscapePluginStream::NetscapePluginStream):
            (WebKit::NetscapePluginStream::sendJavaScriptStream):
            * WebProcess/Plugins/Netscape/NetscapePluginStream.h:
            (WebKit::NetscapePluginStream::create):
            * WebProcess/Plugins/Plugin.h:
            * WebProcess/Plugins/PluginProxy.cpp:
            (WebKit::PluginProxy::didEvaluateJavaScript):
            * WebProcess/Plugins/PluginProxy.h:
            * WebProcess/Plugins/PluginView.cpp:
            (WebKit::PluginView::performJavaScriptURLRequest):
    2011-05-13  Anders Carlsson  <andersca@apple.com>
    
            Reviewed by Sam Weinig.
    
            NPN_GetURL with a javascript: URL doesn't send back the result in the stream
            https://bugs.webkit.org/show_bug.cgi?id=60810
    
            Add a test that calls NPN_GetURL on a javascript: URL and checks that it gets back the expected result.
    
            * plugins/get-url-with-javascript-url-expected.txt: Added.
            * plugins/get-url-with-javascript-url.html: Added.
    2011-05-13  Anders Carlsson  <andersca@apple.com>
    
            Reviewed by Sam Weinig.
    
            NPN_GetURL with a javascript: URL doesn't send back the result in the stream
            https://bugs.webkit.org/show_bug.cgi?id=60810
    
            Add a test that calls NPN_GetURL on a javascript: URL and checks that it gets back the expected result.
    
            * DumpRenderTree/DumpRenderTree.gypi:
            * DumpRenderTree/DumpRenderTree.xcodeproj/project.pbxproj:
            * DumpRenderTree/TestNetscapePlugIn/win/TestNetscapePlugin.vcproj:
            * DumpRenderTree/qt/TestNetscapePlugin/TestNetscapePlugin.pro:
            * GNUmakefile.am:
            Add files.
    
            * DumpRenderTree/TestNetscapePlugIn/PluginTest.cpp:
            * DumpRenderTree/TestNetscapePlugIn/PluginTest.h:
            Add new NPN and NPP wrappers and sort them according to the order they appear in npfunctions.h
    
            * DumpRenderTree/TestNetscapePlugIn/Tests/GetURLWithJavaScriptURL.cpp: Added.
            (GetURLWithJavaScriptURL::GetURLWithJavaScriptURL):
            (GetURLWithJavaScriptURL::NPP_New):
            (GetURLWithJavaScriptURL::NPP_NewStream):
            (GetURLWithJavaScriptURL::NPP_DestroyStream):
            (GetURLWithJavaScriptURL::NPP_WriteReady):
            (GetURLWithJavaScriptURL::NPP_Write):
            * DumpRenderTree/TestNetscapePlugIn/main.cpp:
            (NPP_NewStream):
            (NPP_WriteReady):
            (NPP_Write):
            (NPP_URLNotify):
            call the PluginTest NPP wrappers.
    
    
    git-svn-id: http://svn.webkit.org/repository/webkit/trunk@86467 268f45cc-cd09-0410-ab3c-d52691b4dbfc
    4c8ba961
get-url-with-javascript-url-expected.txt 103 Bytes