Commit 6a9c78e5 authored by ananta@chromium.org's avatar ananta@chromium.org

The webkit java applet creation code passes in the baseURL which is typically...

The webkit java applet creation code passes in the baseURL which is typically the url of the containing document to
create the plugin widget. This URL should not be treated as the plugin SRC url. This mimics the webkit plugin implementation
specifically the upstream WebFrame::createJavaAppletWidget implementation.

This fixes http://code.google.com/p/chromium/issues/detail?id=991, which manifested itself due to our plugin implementation
issuing a spurious GET request for the baseURL which is incorrect as per above.

Bug=991


Review URL: http://codereview.chromium.org/155069

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20005 0039d316-1c4b-4281-b951-d872f2087c98
parent 90194366
......@@ -1419,10 +1419,10 @@ void WebFrameLoaderClient::redirectDataToPlugin(Widget* pluginWidget) {
Widget* WebFrameLoaderClient::createJavaAppletWidget(
const IntSize& size,
HTMLAppletElement* element,
const KURL& url,
const KURL& /* base_url */,
const Vector<String>& param_names,
const Vector<String>& param_values) {
return createPlugin(size, element, url, param_names, param_values,
return createPlugin(size, element, KURL(), param_names, param_values,
"application/x-java-applet", false);
}
......
......@@ -185,7 +185,7 @@ class WebFrameLoaderClient : public WebCore::FrameLoaderClient {
virtual WebCore::Widget* createJavaAppletWidget(
const WebCore::IntSize&,
WebCore::HTMLAppletElement*,
const WebCore::KURL& baseURL,
const WebCore::KURL& /* base_url */,
const WTF::Vector<WebCore::String>& paramNames,
const WTF::Vector<WebCore::String>& paramValues);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment