When the download folder does not exist, change the download folder to a user's "Downloads"

chrome/browser/download/download_manager_unittest.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 <string>
 #include <set>
 
 #include "base/file_util.h"
 #include "base/i18n/number_formatting.h"
 #include "base/i18n/rtl.h"
@@ skipping 113 matching lines @@
   const char* url;
   const char* mime_type;
   bool save_as;
   bool prompt_for_download;
   bool expected_save_as;
 } kStartDownloadCases[] = {
   { "http://www.foo.com/dont-open.html",
     "text/html",
     false,
     false,
-    false, },
+    true, },
   { "http://www.foo.com/save-as.html",
     "text/html",
     true,
     false,
     true, },
   { "http://www.foo.com/always-prompt.html",
     "text/html",
     false,
     true,
     true, },
   { "http://www.foo.com/user-script-text-html-mimetype.user.js",
     "text/html",
     false,
     false,
-    false, },
+    true, },
   { "http://www.foo.com/extensionless-extension",
     "application/x-chrome-extension",
     true,
     false,
     true, },
   { "http://www.foo.com/save-as.pdf",
     "application/pdf",
     true,
     false,
     true, },
   { "http://www.foo.com/sometimes_prompt.pdf",
     "application/pdf",
     false,
     true,
-    false, },
+    true, },
   { "http://www.foo.com/always_prompt.jar",
     "application/jar",
     false,
     true,
     true, },
 };
 
 const struct {
   FilePath::StringType suggested_path;
   bool is_dangerous_file;
@@ skipping 52 matching lines @@
     download_manager_->AddObserver(this);
   }
 
   ~SelectFileObserver() {
     download_manager_->RemoveObserver(this);
   }
 
   // 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);
   }
 
   bool ShowedFileDialogForId(int32 id) {
     return file_dialog_ids_.find(id) != file_dialog_ids_.end();
   }
 
  private:
   std::set<int32> file_dialog_ids_;
   scoped_refptr<DownloadManager> download_manager_;
@@ skipping 59 matching lines @@
     scoped_ptr<DownloadCreateInfo> info(new DownloadCreateInfo);
     info->download_id = static_cast<int>(i);
     info->prompt_user_for_save_location = kStartDownloadCases[i].save_as;
     info->url_chain.push_back(GURL(kStartDownloadCases[i].url));
     info->mime_type = kStartDownloadCases[i].mime_type;
     download_manager_->CreateDownloadItem(info.get());
 
     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();
 
     // SelectFileObserver will have recorded any attempt to open the
     // select file dialog.
     // Note that DownloadManager::FileSelectionCanceled() is never called.
     EXPECT_EQ(kStartDownloadCases[i].expected_save_as,
               observer.ShowedFileDialogForId(i));
   }
 }
@@ skipping 252 matching lines @@
   scoped_ptr<ItemObserver> observer(new ItemObserver(download));
 
   // Create and initialize the download file.  We're bypassing the first part
   // of the download process and skipping to the part after the final file
   // name has been chosen, so we need to initialize the download file
   // properly.
   DownloadFile* download_file(
       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);
 
   ContinueDownloadWithPath(download, new_path);
   message_loop_.RunAllPending();
   EXPECT_TRUE(GetActiveDownloadItem(0) != NULL);
 
   download_file->AppendDataToFile(kTestData, kTestDataLen);
 
   // Finish the download.
@@ skipping 55 matching lines @@
   scoped_ptr<ItemObserver> observer(new ItemObserver(download));
 
   // Create and initialize the download file.  We're bypassing the first part
   // of the download process and skipping to the part after the final file
   // name has been chosen, so we need to initialize the download file
   // properly.
   DownloadFile* download_file(
       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);
 
   ContinueDownloadWithPath(download, new_path);
   message_loop_.RunAllPending();
   EXPECT_TRUE(GetActiveDownloadItem(0) != NULL);
 
   download_file->AppendDataToFile(kTestData, kTestDataLen);
 
   // Finish the download.
@@ skipping 29 matching lines @@
   EXPECT_FALSE(observer->hit_state(DownloadItem::REMOVING));
   EXPECT_TRUE(observer->was_updated());
   EXPECT_FALSE(observer->was_opened());
   EXPECT_TRUE(download->file_externally_removed());
   EXPECT_EQ(DownloadItem::COMPLETE, download->state());
   EXPECT_EQ(download_item_model->GetStatusText(),
             l10n_util::GetStringUTF16(IDS_DOWNLOAD_STATUS_REMOVED));
 
   EXPECT_FALSE(file_util::PathExists(new_path));
 }