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

chrome/browser/download/base_file.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/base_file.h"
 
 #include "base/file_util.h"
 #include "base/format_macros.h"
 #include "base/logging.h"
 #include "base/stringprintf.h"
@@ skipping 28 matching lines @@
 }
 
 BaseFile::~BaseFile() {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::FILE));
   if (detached_)
     Close();
   else
     Cancel();  // Will delete the file.
 }
 
-bool BaseFile::Initialize(bool calculate_hash) {
+bool BaseFile::Initialize(bool calculate_hash, const FilePath& save_path) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::FILE));
   DCHECK(!detached_);
 
   calculate_hash_ = calculate_hash;
 
   if (calculate_hash_)
     secure_hash_.reset(crypto::SecureHash::Create(crypto::SecureHash::SHA256));
 
-  if (!full_path_.empty() ||
-      download_util::CreateTemporaryFileForDownload(&full_path_))
-    return Open();
-  return false;
+  if (full_path_.empty()) {
+    if (!file_util::CreateTemporaryFileInDir(save_path, &full_path_))

[Paweł Hajdan Jr., 2011/06/15 09:31:33]

nit: Instead of having a nested "if", how about adding && !file_util::Create....
to the previous "if"?

[haraken1, 2011/06/15 10:39:45]

Done.

+      return false;
+  }
+  return Open();
 }
 
 bool BaseFile::AppendDataToFile(const char* data, size_t data_len) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::FILE));
   DCHECK(!detached_);
 
   if (!file_stream_.get())
     return false;
 
   // TODO(phajdan.jr): get rid of this check.
@@ skipping 167 matching lines @@
 
 std::string BaseFile::DebugString() const {
   return base::StringPrintf("{ source_url_ = \"%s\""
                             " full_path_ = \"%" PRFilePath "\""
                             " bytes_so_far_ = %" PRId64 " detached_ = %c }",
                             source_url_.spec().c_str(),
                             full_path_.value().c_str(),
                             bytes_so_far_,
                             detached_ ? 'T' : 'F');
 }