Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(77)

Unified Diff: webkit/glue/webplugin_impl.cc

Issue 46026: Get rid of stashing a frame pointer with ResourceRequest and just store the r... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 11 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « webkit/glue/webframeloaderclient_impl.cc ('k') | webkit/glue/weburlrequest.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webkit/glue/webplugin_impl.cc
===================================================================
--- webkit/glue/webplugin_impl.cc (revision 11712)
+++ webkit/glue/webplugin_impl.cc (working copy)
@@ -1241,9 +1241,8 @@
ClientInfo info;
info.id = resource_id;
info.client = client;
- info.request.setFrame(frame());
info.request.setURL(kurl);
- info.request.setOriginPid(delegate_->GetProcessId());
+ info.request.setRequestorProcessID(delegate_->GetProcessId());
info.request.setTargetType(WebCore::ResourceRequest::TargetIsObject);
info.request.setHTTPMethod(method);
@@ -1268,8 +1267,13 @@
SetPostData(&info.request, buf, buf_len);
}
- info.handle = WebCore::ResourceHandle::create(info.request, this, frame(),
- false, false);
+ // Sets the routing id to associate the ResourceRequest with the RenderView.
+ WebCore::ResourceResponse response;
+ frame()->loader()->client()->dispatchWillSendRequest(
+ NULL, 0, info.request, response);
+
+ info.handle = WebCore::ResourceHandle::create(
+ info.request, this, NULL, false, false);
if (!info.handle) {
return false;
}
« no previous file with comments | « webkit/glue/webframeloaderclient_impl.cc ('k') | webkit/glue/weburlrequest.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698