Created a DownloadManager interface, for use in unit tests..

chrome/browser/download/chrome_download_manager_delegate.cc

 // Copyright (c) 2011 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/chrome_download_manager_delegate.h"
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/callback.h"
 #include "base/file_util.h"
@@ skipping 39 matching lines @@
 
 bool ChromeDownloadManagerDelegate::IsExtensionDownload(
     const DownloadItem* item) {
   if (item->prompt_user_for_save_location())
     return false;
 
   return (item->mime_type() == Extension::kMimeType) ||
       UserScript::IsURLUserScript(item->GetURL(), item->mime_type());
 }
 
-void ChromeDownloadManagerDelegate::SetDownloadManager(DownloadManager* dm) {
+void ChromeDownloadManagerDelegate::SetDownloadManager(
+    DownloadManagerInterface* dm) {
   download_manager_ = dm;
   download_history_.reset(new DownloadHistory(profile_));
   download_history_->Load(
-      base::Bind(&DownloadManager::OnPersistentStoreQueryComplete,
+      base::Bind(&DownloadManagerInterface::OnPersistentStoreQueryComplete,
                  base::Unretained(dm)));
 }
 
 void ChromeDownloadManagerDelegate::Shutdown() {
   download_history_.reset();
   download_prefs_.reset();
 }
 
 bool ChromeDownloadManagerDelegate::ShouldStartDownload(int32 download_id) {
   // We create a download item and store it in our download map, and inform the
@@ skipping 306 matching lines @@
         state.prompt_user_for_save_location = true;
     }
     if (download_prefs_->IsDownloadPathManaged()) {
       state.prompt_user_for_save_location = false;
     }
 
     // Determine the proper path for a download, by either one of the following:
     // 1) using the default download directory.
     // 2) prompting the user.
     if (state.prompt_user_for_save_location &&
-        !download_manager_->last_download_path().empty()) {
-      state.suggested_path = download_manager_->last_download_path();
+        !download_manager_->LastDownloadPath().empty()) {
+      state.suggested_path = download_manager_->LastDownloadPath();
     } else {
       state.suggested_path = download_prefs_->download_path();
     }
     state.suggested_path = state.suggested_path.Append(generated_name);
   } else {
     state.suggested_path = state.force_file_name;
   }
 
   if (!state.prompt_user_for_save_location && state.force_file_name.empty()) {
     state.is_dangerous_file =
@@ skipping 159 matching lines @@
 // TODO(noelutz): This function currently works as a callback place holder.
 // Once we decide the hash check is reliable, we could move the
 // MaybeCompleteDownload in OnAllDataSaved to this function.
 void ChromeDownloadManagerDelegate::CheckDownloadHashDone(
     int32 download_id,
     bool is_dangerous_hash) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
   DVLOG(1) << "CheckDownloadHashDone, download_id: " << download_id
            << " is dangerous_hash: " << is_dangerous_hash;
 }