Made ownership of DownloadRequestHandles clear through scoped_ptr<>

content/browser/download/download_manager_impl_unittest.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 <set>
 #include <string>
 
 #include "base/bind.h"
 #include "base/file_util.h"
 #include "base/memory/scoped_ptr.h"
@@ skipping 138 matching lines @@
 
   // Overridden methods from DownloadItemFactory.
   virtual content::DownloadItem* CreatePersistedItem(
       DownloadItemImpl::Delegate* delegate,
       content::DownloadId download_id,
       const content::DownloadPersistentStoreInfo& info,
       const net::BoundNetLog& bound_net_log) OVERRIDE;
   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;
   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 43 matching lines @@
   EXPECT_CALL(*result, GetId())
       .WillRepeatedly(Return(local_id));
   items_[local_id] = result;
 
   return result;
 }
 
 content::DownloadItem* MockDownloadItemFactory::CreateActiveItem(
     DownloadItemImpl::Delegate* delegate,
     const DownloadCreateInfo& info,
-    DownloadRequestHandleInterface* request_handle,
+    scoped_ptr<DownloadRequestHandleInterface> request_handle,
     bool is_otr,
     const net::BoundNetLog& bound_net_log) {
   int local_id = info.download_id.local();
   DCHECK(items_.find(local_id) == items_.end());
 
   content::MockDownloadItem* result =
       new StrictMock<content::MockDownloadItem>;
   EXPECT_CALL(*result, GetId())
       .WillRepeatedly(Return(local_id));
   items_[local_id] = result;
@@ skipping 334 matching lines @@
               UpdateItemInPersistentStore(&item));
   EXPECT_CALL(item, GetState())
       .WillRepeatedly(Return(DownloadItem::CANCELLED));
   EXPECT_CALL(item, GetDbHandle())
       .WillRepeatedly(Return(db_handle));
 
   EXPECT_CALL(item, OffThreadCancel(&GetMockDownloadFileManager()));
   download_manager_->DownloadCancelled(&item);
   EXPECT_EQ(NULL, GetActiveDownloadItem(download_id));
 }