diff -r d39add9822e2 -r 5bfc169077b2 webengine/osswebengine/WebKit/s60/plugins/PluginStreamLoaderClient.cpp --- a/webengine/osswebengine/WebKit/s60/plugins/PluginStreamLoaderClient.cpp Tue Feb 02 00:56:45 2010 +0200 +++ b/webengine/osswebengine/WebKit/s60/plugins/PluginStreamLoaderClient.cpp Fri Mar 19 09:52:28 2010 +0200 @@ -36,36 +36,33 @@ using namespace WebCore; -NetscapePlugInStreamLoaderClient* NetscapePlugInStreamLoaderClient::NewL(const String& url, PluginSkin* pluginskin, Frame* frame, void* notifydata, TBool notify/*=EFalse*/) +NetscapePlugInStreamLoaderClient* NetscapePlugInStreamLoaderClient::NewL(const String& url, PluginSkin* pluginskin, Frame* frame, void* notifydata) { NetscapePlugInStreamLoaderClient* self = new (ELeave) NetscapePlugInStreamLoaderClient(); CleanupStack::PushL( self ); - self->ConstructL( url, pluginskin, frame, notifydata, notify ); + self->ConstructL( url, pluginskin, frame, notifydata ); CleanupStack::Pop(); return self; } -NetscapePlugInStreamLoaderClient* NetscapePlugInStreamLoaderClient::NewL(const ResourceRequest& request, PluginSkin* pluginskin, Frame* frame, void* notifydata, TBool notify/*=EFalse*/) +NetscapePlugInStreamLoaderClient* NetscapePlugInStreamLoaderClient::NewL(const ResourceRequest& request, PluginSkin* pluginskin, Frame* frame, void* notifydata) { NetscapePlugInStreamLoaderClient* self = new (ELeave) NetscapePlugInStreamLoaderClient(); CleanupStack::PushL( self ); - self->ConstructL( request, pluginskin, frame, notifydata, notify ); + self->ConstructL( request, pluginskin, frame, notifydata ); CleanupStack::Pop(); return self; } -void NetscapePlugInStreamLoaderClient::ConstructL(const String& url, PluginSkin* pluginskin, Frame* frame, void* notifydata, TBool notify/*=EFalse*/) +void NetscapePlugInStreamLoaderClient::ConstructL(const String& url, PluginSkin* pluginskin, Frame* frame, void* notifydata) { m_loader = 0; m_request = 0; m_pluginstream = 0; - m_pluginskin = pluginskin; - m_notifydata = notifydata; m_frame = frame; - m_notify = notify; m_pluginstream = new (ELeave) PluginStream(pluginskin, this, notifydata); m_request = new (ELeave) ResourceRequest(m_frame->loader()->completeURL(url)); @@ -81,16 +78,13 @@ m_loader->setShouldBufferData(false); } -void NetscapePlugInStreamLoaderClient::ConstructL(const ResourceRequest& request, PluginSkin* pluginskin, Frame* frame, void* notifydata, TBool notify/*=EFalse*/) +void NetscapePlugInStreamLoaderClient::ConstructL(const ResourceRequest& request, PluginSkin* pluginskin, Frame* frame, void* notifydata) { m_loader = 0; m_request = 0; m_pluginstream = 0; - m_pluginskin = pluginskin; - m_notifydata = notifydata; m_frame = frame; - m_notify = notify; m_pluginstream = new (ELeave) PluginStream(pluginskin, this, notifydata); m_request = new (ELeave) ResourceRequest(request.url()); @@ -130,8 +124,6 @@ delete m_request; delete m_pluginstream; - m_pluginskin = NULL; - m_notifydata = NULL; } @@ -155,10 +147,9 @@ void NetscapePlugInStreamLoaderClient::cancelWithError(const ResourceError& error) { - if (m_loader && !m_loader->isDone()){ - m_loader->cancel(error); + if (m_loader && !m_loader->isDone()) + m_loader->cancel(error); } -} void NetscapePlugInStreamLoaderClient::didReceiveResponse(const ResourceResponse& response) { @@ -200,22 +191,14 @@ void NetscapePlugInStreamLoaderClient::didFail(const ResourceError& error) { if (m_pluginstream) { - int err = error.errorCode() ? error.errorCode() : KErrCancel; - HBufC* failedURL = HBufC::NewLC(error.failingURL().length()); - failedURL->Des().Copy(error.failingURL()); - m_pluginstream->destroyStream(err, failedURL); - CleanupStack::PopAndDestroy(failedURL); + m_pluginstream->destroyStream(error.errorCode() ? error.errorCode() : KErrCancel); } } void NetscapePlugInStreamLoaderClient::didCancel(const ResourceError& error) { if (m_pluginstream) { - int err = error.errorCode() ? error.errorCode() : KErrCancel; - HBufC* failedURL = HBufC::NewLC(error.failingURL().length()); - failedURL->Des().Copy(error.failingURL()); - m_pluginstream->destroyStream(err, failedURL); - CleanupStack::PopAndDestroy(failedURL); + m_pluginstream->destroyStream(error.errorCode() ? error.errorCode() : KErrCancel); } }