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

Unified Diff: content/browser/download/download_manager_impl.cc

Issue 9150028: Cleanup: ResourceDispatcherHost::BeginDownload takes a scoped_ptr<URLRequest> argument. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: s/process_unique_id/child_id/ Created 8 years, 11 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
Index: content/browser/download/download_manager_impl.cc
diff --git a/content/browser/download/download_manager_impl.cc b/content/browser/download/download_manager_impl.cc
index c6b7de59274381b5719efe5cc8d7397800038b81..a5e1c05ba5386eaf9a4179ea8d75661587f03ea3 100644
--- a/content/browser/download/download_manager_impl.cc
+++ b/content/browser/download/download_manager_impl.cc
@@ -73,11 +73,11 @@ void BeginDownload(const URLParams& url_params,
ResourceDispatcherHost* resource_dispatcher_host,
const RenderParams& render_params,
const content::ResourceContext* context) {
- net::URLRequest* request = new net::URLRequest(url_params.url_,
- resource_dispatcher_host);
+ scoped_ptr<net::URLRequest> request(
+ new net::URLRequest(url_params.url_, resource_dispatcher_host));
request->set_referrer(url_params.referrer_.spec());
resource_dispatcher_host->BeginDownload(
- request, save_info, true,
+ request.Pass(), save_info, true,
DownloadResourceHandler::OnStartedCallback(),
render_params.render_process_id_,
render_params.render_view_id_,
« no previous file with comments | « chrome/browser/download/download_extension_api.cc ('k') | content/browser/renderer_host/render_message_filter.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698