Revert 91861 - When the download folder does not exist, change the download folder to a user's "D...

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 217 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,
-                                         const FilePath& suggested_path) {
+  virtual void SelectFileDialogDisplayed(int32 id) {
     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 36 matching lines @@
   DownloadItem* tracked_;
   int states_hit_;
   bool was_updated_;
   bool was_opened_;
 };
 
 }  // namespace
 
 TEST_F(DownloadManagerTest, StartDownload) {
   BrowserThread io_thread(BrowserThread::IO, &message_loop_);
-  ScopedTempDir download_save_dir;
-  ASSERT_TRUE(download_save_dir.CreateUniqueTempDir());
   PrefService* prefs = profile_->GetPrefs();
-  prefs->SetFilePath(prefs::kDownloadDefaultDirectory,
-                     download_save_dir.path());
+  prefs->SetFilePath(prefs::kDownloadDefaultDirectory, FilePath());
   download_manager_->download_prefs()->EnableAutoOpenBasedOnExtension(
       FilePath(FILE_PATH_LITERAL("example.pdf")));
 
   for (size_t i = 0; i < ARRAYSIZE_UNSAFE(kStartDownloadCases); ++i) {
     prefs->SetBoolean(prefs::kPromptForDownload,
                       kStartDownloadCases[i].prompt_for_download);
 
     SelectFileObserver observer(download_manager_);
     // Normally, the download system takes ownership of info, and is
     // responsible for deleting it.  In these unit tests, however, we
@@ skipping 401 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));
 }