BrowserContext should simply own DownloadManager

content/public/test/test_file_error_injector.h

 // 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.
 
 #ifndef CONTENT_PUBLIC_TEST_TEST_FILE_ERROR_INJECTOR_H_
 #define CONTENT_PUBLIC_TEST_TEST_FILE_ERROR_INJECTOR_H_
 
 #include <map>
 #include <set>
 #include <string>
@@ skipping 56 matching lines @@
     DownloadInterruptReason error;  // Error to inject.
   };
 
   typedef std::map<std::string, FileErrorInfo> ErrorMap;
 
   // Creates an instance.  May only be called once.
   // Lives until all callbacks (in the implementation) are complete and the
   // 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.
   // It is an error to call |AddError()| more than once for the same file
   // (URL), unless you call |ClearErrors()| in between them.
   bool AddError(const FileErrorInfo& error_info);
 
   // Clears all errors.
   // Only affects files created after the next call to InjectErrors().
   void ClearErrors();
@@ skipping 17 matching lines @@
   // Resets the found file list.
   void ClearFoundFiles();
 
   static std::string DebugString(FileOperationCode code);
 
  private:
   friend class base::RefCountedThreadSafe<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();
 
   // Callbacks from the download file, to record lifetimes.
   // These may be called on any thread.
   void RecordDownloadFileConstruction(const GURL& url);
   void RecordDownloadFileDestruction(const GURL& url);
 
   // These run on the UI thread.
   void DownloadFileCreated(GURL url);
   void DestroyingDownloadFile(GURL url);
 
   // All the data is used on the UI thread.
   // Our injected error list, mapped by URL.  One per file.
   ErrorMap injected_errors_;
 
   // Keep track of active DownloadFiles.
   FileSet files_;
 
   // Keep track of found DownloadFiles.
   FileSet found_files_;
 
   // The factory we created.  May outlive this class.
   DownloadFileWithErrorsFactory* created_factory_;
 
   // The download manager we set the factory on.
-  scoped_refptr<DownloadManagerImpl> download_manager_;
+  DownloadManagerImpl* download_manager_;
 
   DISALLOW_COPY_AND_ASSIGN(TestFileErrorInjector);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_TEST_TEST_FILE_ERROR_INJECTOR_H_