Added a download file factory to the download file manager, for testing.

chrome/browser/download/download_manager_unittest.cc

Index: chrome/browser/download/download_manager_unittest.cc
diff --git a/chrome/browser/download/download_manager_unittest.cc b/chrome/browser/download/download_manager_unittest.cc
index b99007e9bd0fd8b4d53c9e1ec71fffb78a3d667c..8fc960cd47e489a4df66cc3fe4fb6818403da416 100644
--- a/chrome/browser/download/download_manager_unittest.cc
+++ b/chrome/browser/download/download_manager_unittest.cc
@@ -149,7 +149,7 @@ class DownloadManagerTest : public testing::Test {
 
   DownloadFileManager* file_manager() {
     if (!file_manager_) {
-      file_manager_ = new DownloadFileManager(NULL);
+      file_manager_ = new DownloadFileManager(NULL, NULL);

[Randy Smith (Not in Mondays), 2011/12/05 22:25:38]

You should either specify what happens when you pass in a NULL at the
constructor declaration, or (preferred) throw together a quick mock here that
asserts when the factor GetFile is called.

[ahendrickson, 2011/12/07 19:55:15]

Did both!

Using a NULL to indicate that the default DownloadFileFactory should be used.

       download_manager_->SetFileManager(file_manager_);
     }
     return file_manager_;