| Index: content/renderer/npapi/webplugin_impl.cc
|
| diff --git a/content/renderer/npapi/webplugin_impl.cc b/content/renderer/npapi/webplugin_impl.cc
|
| index ac739b5bc974523564a2ec6ba1dec844a03e7e65..fe30ce86cc5e80735e74b784269cfb0824cdd41d 100644
|
| --- a/content/renderer/npapi/webplugin_impl.cc
|
| +++ b/content/renderer/npapi/webplugin_impl.cc
|
| @@ -463,31 +463,6 @@ void WebPluginImpl::didFailLoading(const WebURLError& error) {
|
| delegate_->DidManualLoadFail();
|
| }
|
|
|
| -void WebPluginImpl::didFinishLoadingFrameRequest(
|
| - const WebURL& url, void* notify_data) {
|
| - if (delegate_) {
|
| - // We're converting a void* into an arbitrary int id. Though
|
| - // these types are the same size on all the platforms we support,
|
| - // the compiler may complain as though they are different, so to
|
| - // make the casting gods happy go through an intptr_t (the union
|
| - // of void* and int) rather than converting straight across.
|
| - delegate_->DidFinishLoadWithReason(
|
| - url, NPRES_DONE, reinterpret_cast<intptr_t>(notify_data));
|
| - }
|
| -}
|
| -
|
| -void WebPluginImpl::didFailLoadingFrameRequest(
|
| - const WebURL& url, void* notify_data, const WebURLError& error) {
|
| - if (!delegate_)
|
| - return;
|
| -
|
| - NPReason reason =
|
| - error.reason == net::ERR_ABORTED ? NPRES_USER_BREAK : NPRES_NETWORK_ERR;
|
| - // See comment in didFinishLoadingFrameRequest about the cast here.
|
| - delegate_->DidFinishLoadWithReason(
|
| - url, reason, reinterpret_cast<intptr_t>(notify_data));
|
| -}
|
| -
|
| bool WebPluginImpl::isPlaceholder() {
|
| return false;
|
| }
|
| @@ -697,7 +672,6 @@ WebPluginImpl::RoutingStatus WebPluginImpl::RouteToFrame(
|
| const char* target,
|
| const char* buf,
|
| unsigned int len,
|
| - int notify_id,
|
| ReferrerValue referrer_flag) {
|
| // If there is no target, there is nothing to do
|
| if (!target)
|
| @@ -761,8 +735,7 @@ WebPluginImpl::RoutingStatus WebPluginImpl::RouteToFrame(
|
| }
|
| }
|
|
|
| - container_->loadFrameRequest(
|
| - request, target_str, notify_id != 0, reinterpret_cast<void*>(notify_id));
|
| + container_->loadFrameRequest(request, target_str);
|
| return ROUTED;
|
| }
|
|
|
| @@ -902,7 +875,7 @@ void WebPluginImpl::InvalidateRect(const gfx::Rect& rect) {
|
|
|
| void WebPluginImpl::OnDownloadPluginSrcUrl() {
|
| HandleURLRequestInternal(
|
| - plugin_url_.spec().c_str(), "GET", NULL, NULL, 0, 0, false, DOCUMENT_URL,
|
| + plugin_url_.spec().c_str(), "GET", NULL, NULL, 0, false, DOCUMENT_URL,
|
| false, true);
|
| }
|
|
|
| @@ -1038,7 +1011,7 @@ void WebPluginImpl::didReceiveResponse(WebURLLoader* loader,
|
| for (size_t i = 0; i < clients_.size(); ++i) {
|
| if (clients_[i].loader.get() == loader) {
|
| WebPluginResourceClient* resource_client =
|
| - delegate_->CreateResourceClient(clients_[i].id, plugin_url_, 0);
|
| + delegate_->CreateResourceClient(clients_[i].id, plugin_url_);
|
| clients_[i].client = resource_client;
|
| client = resource_client;
|
| break;
|
| @@ -1163,13 +1136,12 @@ void WebPluginImpl::HandleURLRequest(const char* url,
|
| const char* target,
|
| const char* buf,
|
| unsigned int len,
|
| - int notify_id,
|
| bool popups_allowed,
|
| bool notify_redirects) {
|
| // GetURL/PostURL requests initiated explicitly by plugins should specify the
|
| // plugin SRC url as the referrer if it is available.
|
| HandleURLRequestInternal(
|
| - url, method, target, buf, len, notify_id, popups_allowed, PLUGIN_SRC,
|
| + url, method, target, buf, len, popups_allowed, PLUGIN_SRC,
|
| notify_redirects, false);
|
| }
|
|
|
| @@ -1178,7 +1150,6 @@ void WebPluginImpl::HandleURLRequestInternal(const char* url,
|
| const char* target,
|
| const char* buf,
|
| unsigned int len,
|
| - int notify_id,
|
| bool popups_allowed,
|
| ReferrerValue referrer_flag,
|
| bool notify_redirects,
|
| @@ -1194,7 +1165,7 @@ void WebPluginImpl::HandleURLRequestInternal(const char* url,
|
| url::FindAndCompareScheme(url, strlen(url), url::kJavaScriptScheme, NULL);
|
| RoutingStatus routing_status = RouteToFrame(
|
| url, is_javascript_url, popups_allowed, method, target, buf, len,
|
| - notify_id, referrer_flag);
|
| + referrer_flag);
|
| if (routing_status == ROUTED)
|
| return;
|
|
|
| @@ -1205,8 +1176,7 @@ void WebPluginImpl::HandleURLRequestInternal(const char* url,
|
| // delegate_ could be NULL because executeScript caused the container to
|
| // be deleted.
|
| if (delegate_) {
|
| - delegate_->SendJavaScriptStream(
|
| - gurl, result.utf8(), !result.isNull(), notify_id);
|
| + delegate_->SendJavaScriptStream(gurl, result.utf8(), !result.isNull());
|
| }
|
|
|
| return;
|
| @@ -1226,7 +1196,7 @@ void WebPluginImpl::HandleURLRequestInternal(const char* url,
|
| if ((routing_status == INVALID_URL) ||
|
| (routing_status == GENERAL_FAILURE)) {
|
| WebPluginResourceClient* resource_client = delegate_->CreateResourceClient(
|
| - resource_id, complete_url, notify_id);
|
| + resource_id, complete_url);
|
| if (resource_client)
|
| resource_client->DidFail(resource_id);
|
| return;
|
| @@ -1257,14 +1227,14 @@ void WebPluginImpl::HandleURLRequestInternal(const char* url,
|
| request.referrerPolicy());
|
|
|
| GURL first_party_for_cookies = webframe_->document().firstPartyForCookies();
|
| - delegate_->FetchURL(resource_id, notify_id, complete_url,
|
| + delegate_->FetchURL(resource_id, complete_url,
|
| first_party_for_cookies, method, buf, len, referrer,
|
| notify_redirects, is_plugin_src_load, 0,
|
| render_frame_->GetRoutingID(),
|
| render_view_->GetRoutingID());
|
| } else {
|
| WebPluginResourceClient* resource_client = delegate_->CreateResourceClient(
|
| - resource_id, complete_url, notify_id);
|
| + resource_id, complete_url);
|
| if (!resource_client)
|
| return;
|
| InitiateHTTPRequest(resource_id, resource_client, complete_url, method, buf,
|
|
|