BrowserContext should simply own DownloadManager

chrome/browser/download/download_ui_controller_unittest.cc

 // Copyright 2013 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/files/file_path.h"
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/memory/weak_ptr.h"
 #include "chrome/browser/download/download_ui_controller.h"
 #include "content/public/test/mock_download_item.h"
@@ skipping 51 matching lines @@
   scoped_ptr<DownloadUIController::Delegate> GetTestDelegate();
 
   MockDownloadManager* manager() { return manager_.get(); }
   content::DownloadManager::Observer* manager_observer() {
     return manager_observer_;
   }
   content::DownloadItem::Observer* item_observer() { return item_observer_; }
   content::DownloadItem* received_item() { return received_item_; }
 
  private:
-  scoped_refptr<MockDownloadManager> manager_;
+  scoped_ptr<MockDownloadManager> manager_;
   content::DownloadManager::Observer* manager_observer_;
   content::DownloadItem::Observer* item_observer_;
   content::DownloadItem* received_item_;
 
   base::WeakPtrFactory<content::DownloadItem*> receiver_factory_;
 };
 
 DownloadUIControllerTest::DownloadUIControllerTest()
     : manager_observer_(NULL),
       item_observer_(NULL),
       received_item_(NULL),
       receiver_factory_(&received_item_) {
 }
 
 void DownloadUIControllerTest::SetUp() {
-  manager_ = new testing::StrictMock<MockDownloadManager>();
+  manager_.reset(new testing::StrictMock<MockDownloadManager>());
   EXPECT_CALL(*manager_, AddObserver(_))
       .WillOnce(SaveArg<0>(&manager_observer_));
   EXPECT_CALL(*manager_, RemoveObserver(_))
       .WillOnce(Assign(&manager_observer_,
                        static_cast<content::DownloadManager::Observer*>(NULL)));
   EXPECT_CALL(*manager_, GetAllDownloads(_));
 }
 
 scoped_ptr<MockDownloadItem> DownloadUIControllerTest::GetMockDownload() {
   scoped_ptr<MockDownloadItem> item(
@@ skipping 75 matching lines @@
 
   ASSERT_TRUE(manager_observer());
   manager_observer()->OnDownloadCreated(manager(), item.get());
   EXPECT_FALSE(received_item());
 
   item_observer()->OnDownloadUpdated(item.get());
   EXPECT_FALSE(received_item());
 }
 
 } // namespace