BrowserContext should simply own DownloadManager

content/public/test/test_file_error_injector.cc

Index: content/public/test/test_file_error_injector.cc
diff --git a/content/public/test/test_file_error_injector.cc b/content/public/test/test_file_error_injector.cc
index be02cc05876606627143bb5f83f7185b6e81d6c3..c2a6fc3b7ee306fed1a8bf17f502c87b200fd6a4 100644
--- a/content/public/test/test_file_error_injector.cc
+++ b/content/public/test/test_file_error_injector.cc
@@ -338,12 +338,11 @@ void DownloadFileWithErrorsFactory::ClearErrors() {
 }
 
 TestFileErrorInjector::TestFileErrorInjector(
-    scoped_refptr<DownloadManager> download_manager)
+    DownloadManager* download_manager)
     : created_factory_(NULL),
       // This code is only used for browser_tests, so a
       // DownloadManager is always a DownloadManagerImpl.
-      download_manager_(
-          static_cast<DownloadManagerImpl*>(download_manager.get())) {
+      download_manager_(static_cast<DownloadManagerImpl*>(download_manager)) {
   // Record the value of the pointer, for later validation.
   created_factory_ =
       new DownloadFileWithErrorsFactory(
@@ -446,7 +445,7 @@ void TestFileErrorInjector::RecordDownloadFileDestruction(const GURL& url) {
 
 // static
 scoped_refptr<TestFileErrorInjector> TestFileErrorInjector::Create(
-    scoped_refptr<DownloadManager> download_manager) {
+    DownloadManager* download_manager) {
   static bool visited = false;
   DCHECK(!visited);  // Only allowed to be called once.
   visited = true;