BrowserContext should simply own DownloadManager

content/public/test/test_file_error_injector.h

Index: content/public/test/test_file_error_injector.h
diff --git a/content/public/test/test_file_error_injector.h b/content/public/test/test_file_error_injector.h
index 38ebf6368e58a829f6a2bbacea4f9b5cb58eb133..37c4c8edd4e416b91b36b90735213c091faae9e6 100644
--- a/content/public/test/test_file_error_injector.h
+++ b/content/public/test/test_file_error_injector.h
@@ -74,7 +74,7 @@ class TestFileErrorInjector
   // creator goes out of scope.
   // TODO(rdsmith): Allow multiple calls for different download managers.
   static scoped_refptr<TestFileErrorInjector> Create(
-      scoped_refptr<DownloadManager> download_manager);
+      DownloadManager* download_manager);
 
   // Adds an error.
   // Must be called before |InjectErrors()| for a particular download file.
@@ -112,8 +112,7 @@ class TestFileErrorInjector
 
   typedef std::set<GURL> FileSet;
 
-  TestFileErrorInjector(
-      scoped_refptr<DownloadManager> download_manager);
+  TestFileErrorInjector(DownloadManager* download_manager);

[sky, 2013/05/23 21:55:53]

explicit

 
   virtual ~TestFileErrorInjector();
 
@@ -140,7 +139,7 @@ class TestFileErrorInjector
   DownloadFileWithErrorsFactory* created_factory_;
 
   // The download manager we set the factory on.
-  scoped_refptr<DownloadManagerImpl> download_manager_;
+  DownloadManagerImpl* download_manager_;
 
   DISALLOW_COPY_AND_ASSIGN(TestFileErrorInjector);
 };