Made ownership of DownloadRequestHandles clear through scoped_ptr<>

content/browser/download/download_manager_impl.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_manager_impl.h"
 
 #include <iterator>
 
 #include "base/bind.h"
 #include "base/callback.h"
@@ skipping 134 matching lines @@
         DownloadItemImpl::Delegate* delegate,
         content::DownloadId download_id,
         const content::DownloadPersistentStoreInfo& info,
         const net::BoundNetLog& bound_net_log) OVERRIDE {
       return new DownloadItemImpl(delegate, download_id, info, bound_net_log);
     }
 
     virtual content::DownloadItem* CreateActiveItem(
         DownloadItemImpl::Delegate* delegate,
         const DownloadCreateInfo& info,
-        DownloadRequestHandleInterface* request_handle,
+        scoped_ptr<DownloadRequestHandleInterface> request_handle,
         bool is_otr,
         const net::BoundNetLog& bound_net_log) OVERRIDE {
-      return new DownloadItemImpl(delegate, info, request_handle, is_otr,
-                                  bound_net_log);
+      return new DownloadItemImpl(delegate, info, request_handle.Pass(),
+                                  is_otr, bound_net_log);
     }
 
     virtual content::DownloadItem* CreateSavePageItem(
         DownloadItemImpl::Delegate* delegate,
         const FilePath& path,
         const GURL& url,
         bool is_otr,
         content::DownloadId download_id,
         const std::string& mime_type,
         const net::BoundNetLog& bound_net_log) OVERRIDE {
@@ skipping 283 matching lines @@
 
 net::BoundNetLog DownloadManagerImpl::CreateDownloadItem(
     DownloadCreateInfo* info, const DownloadRequestHandle& request_handle) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
 
   net::BoundNetLog bound_net_log =
       net::BoundNetLog::Make(net_log_, net::NetLog::SOURCE_DOWNLOAD);
   if (!info->download_id.IsValid())
     info->download_id = GetNextId();
   DownloadItem* download = factory_->CreateActiveItem(
-      this, *info, new DownloadRequestHandle(request_handle),
+      this, *info, scoped_ptr<DownloadRequestHandleInterface>(
+          new DownloadRequestHandle(request_handle)).Pass(),
       browser_context_->IsOffTheRecord(), bound_net_log);
   int32 download_id = info->download_id.local();
 
   DCHECK(!ContainsKey(active_downloads_, download_id));
   downloads_.insert(download);
   active_downloads_[download_id] = download;
 
   return bound_net_log;
 }
 
@@ skipping 702 matching lines @@
 void DownloadManagerImpl::DownloadRenamedToFinalName(
     DownloadItem* download) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
   // If the rename failed, we receive an OnDownloadInterrupted() call before we
   // receive the DownloadRenamedToFinalName() call.
   if (delegate_) {
     delegate_->UpdatePathForItemInPersistentStore(
         download, download->GetFullPath());
   }
 }