Shift passage of FileStream in downloads system to be by scoped_ptr<>.

content/browser/download/download_file_factory.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/download/download_file_factory.h"
 
 #include "content/browser/download/download_file_impl.h"
 #include "content/browser/power_save_blocker.h"
 
 namespace content {
 
 DownloadFileFactory::~DownloadFileFactory() {}
 
 DownloadFile* DownloadFileFactory::CreateFile(
-    DownloadCreateInfo* info,
+    scoped_ptr<DownloadCreateInfo> info,
     scoped_ptr<content::ByteStreamReader> stream,
     DownloadManager* download_manager,
     bool calculate_hash,
     const net::BoundNetLog& bound_net_log) {
+  // Ownership will be taken by DownloadFileImpl.

[benjhayden, 2012/10/11 19:41:12]

So why not make DFI take a scoped_ptr?

[Randy Smith (Not in Mondays), 2012/10/15 18:56:32]

You know, as I went through this CL, I *knew* you were going to ask that :-}. 
Trying to minimize changes, but you're right that that's the right thing.  Done.

+  DownloadRequestHandle* request_handle(
+      new DownloadRequestHandle(info->request_handle));
+
   return new DownloadFileImpl(
-      info, stream.Pass(), new DownloadRequestHandle(info->request_handle),
-      download_manager, calculate_hash,
+      info.Pass(), stream.Pass(), request_handle, download_manager,
+      calculate_hash,
       scoped_ptr<content::PowerSaveBlocker>(
           new content::PowerSaveBlocker(
               content::PowerSaveBlocker::kPowerSaveBlockPreventAppSuspension,
               "Download in progress")).Pass(),
       bound_net_log);
 }
 
 }  // namespace content