BrowserContext should simply own DownloadManager

chrome/browser/download/all_download_item_notifier_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 "chrome/browser/download/all_download_item_notifier.h"
 
 #include "content/public/test/mock_download_item.h"
 #include "content/public/test/mock_download_manager.h"
 #include "testing/gmock/include/gmock/gmock.h"
 #include "testing/gtest/include/gtest/gtest.h"
@@ skipping 52 matching lines @@
     notifier_.reset(new AllDownloadItemNotifier(
         download_manager_.get(), &observer_));
   }
 
   void ClearNotifier() {
     notifier_.reset();
   }
 
  private:
   NiceMock<content::MockDownloadItem> item_;
-  scoped_refptr<content::MockDownloadManager> download_manager_;
+  scoped_ptr<content::MockDownloadManager> download_manager_;
   scoped_ptr<AllDownloadItemNotifier> notifier_;
   NiceMock<MockNotifierObserver> observer_;
 
   DISALLOW_COPY_AND_ASSIGN(AllDownloadItemNotifierTest);
 };
 
 }  // namespace
 
 TEST_F(AllDownloadItemNotifierTest,
     AllDownloadItemNotifierTest_0) {
@@ skipping 29 matching lines @@
       &manager(), &item());
 
   EXPECT_CALL(manager(), RemoveObserver(NotifierAsManagerObserver()));
   NotifierAsManagerObserver()->ManagerGoingDown(&manager());
 
   EXPECT_CALL(item(), RemoveObserver(NotifierAsItemObserver()));
   NotifierAsItemObserver()->OnDownloadDestroyed(&item());
 
   ClearNotifier();
 }