| 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 0e947d1325432e5c7ebf5a7dc6cfca356b075ee4..65bcd8c23db391f2d563d76488368dc30c08fe86 100644
|
| --- a/chrome/browser/download/download_manager_unittest.cc
|
| +++ b/chrome/browser/download/download_manager_unittest.cc
|
| @@ -124,7 +124,7 @@ const struct {
|
| "text/html",
|
| false,
|
| false,
|
| - false, },
|
| + true, },
|
| { "http://www.foo.com/save-as.html",
|
| "text/html",
|
| true,
|
| @@ -139,7 +139,7 @@ const struct {
|
| "text/html",
|
| false,
|
| false,
|
| - false, },
|
| + true, },
|
| { "http://www.foo.com/extensionless-extension",
|
| "application/x-chrome-extension",
|
| true,
|
| @@ -154,7 +154,7 @@ const struct {
|
| "application/pdf",
|
| false,
|
| true,
|
| - false, },
|
| + true, },
|
| { "http://www.foo.com/always_prompt.jar",
|
| "application/jar",
|
| false,
|
| @@ -227,7 +227,8 @@ class SelectFileObserver : public DownloadManager::Observer {
|
| // Downloadmanager::Observer functions.
|
| virtual void ModelChanged() {}
|
| virtual void ManagerGoingDown() {}
|
| - virtual void SelectFileDialogDisplayed(int32 id) {
|
| + virtual void SelectFileDialogDisplayed(int32 id,
|
| + const FilePath& suggested_path) {
|
| file_dialog_ids_.insert(id);
|
| }
|
|
|
| @@ -307,7 +308,10 @@ TEST_F(DownloadManagerTest, StartDownload) {
|
| DownloadFile* download_file(
|
| new DownloadFile(info.get(), download_manager_));
|
| AddDownloadToFileManager(info->download_id, download_file);
|
| - download_file->Initialize(false);
|
| +
|
| + ScopedTempDir temp_dir;
|
| + ASSERT_TRUE(temp_dir.CreateUniqueTempDir());
|
| + download_file->Initialize(false, temp_dir.path());
|
| download_manager_->StartDownload(info->download_id);
|
| message_loop_.RunAllPending();
|
|
|
| @@ -556,7 +560,9 @@ TEST_F(DownloadManagerTest, DownloadOverwriteTest) {
|
| new DownloadFile(info.get(), download_manager_));
|
| download_file->Rename(cr_path);
|
| // This creates the .crdownload version of the file.
|
| - download_file->Initialize(false);
|
| + ScopedTempDir temp_dir;
|
| + ASSERT_TRUE(temp_dir.CreateUniqueTempDir());
|
| + download_file->Initialize(false, temp_dir.path());
|
| // |download_file| is owned by DownloadFileManager.
|
| AddDownloadToFileManager(info->download_id, download_file);
|
|
|
|
|