Made ownership of DownloadRequestHandles clear through scoped_ptr<>

content/browser/download/download_item_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 "base/message_loop.h"
 #include "base/stl_util.h"
 #include "base/threading/thread.h"
 #include "content/browser/download/byte_stream.h"
 #include "content/browser/download/download_create_info.h"
 #include "content/browser/download/download_file_manager.h"
@@ skipping 167 matching lines @@
     scoped_ptr<DownloadCreateInfo> info_;
 
     info_.reset(new DownloadCreateInfo());
     static int next_id;
     info_->download_id =
         content::DownloadId(kValidDownloadItemIdDomain, ++next_id);
     info_->prompt_user_for_save_location = false;
     info_->url_chain.push_back(GURL());
     info_->state = state;
 
-    MockRequestHandle* request_handle =
-        new testing::NiceMock<MockRequestHandle>;
+    scoped_ptr<DownloadRequestHandleInterface> request_handle(
+        new testing::NiceMock<MockRequestHandle>);
     DownloadItem* download =
         new DownloadItemImpl(&delegate_, *(info_.get()),
-                             request_handle, false, net::BoundNetLog());
+                             request_handle.Pass(), false, net::BoundNetLog());
     allocated_downloads_.insert(download);
     return download;
   }
 
   // Destroy a previously created download item.
   void DestroyDownloadItem(DownloadItem* item) {
     allocated_downloads_.erase(item);
     delete item;
   }
 
@@ skipping 385 matching lines @@
   DownloadItem* item = CreateDownloadItem(DownloadItem::IN_PROGRESS);
 
   EXPECT_EQ(false, item->GetFileExternallyRemoved());
   item->OnDownloadedFileRemoved();
   EXPECT_EQ(true, item->GetFileExternallyRemoved());
 }
 
 TEST(MockDownloadItem, Compiles) {
   MockDownloadItem mock_item;
 }