Change to make the 'save as...' dialog choose the default download path you have selected in your prefs instead of always choosing the chrome::DIR_DEFAULT_DOWNLOADS path.

This will only affect you if you have not used 'save as...' before and do not have a SaveFileDefaultDirectory set.

Patch contributed by Paul Robinson <paulRobinson85@googlemail.com>

BUG=8049
Review URL: http://codereview.chromium.org/43016

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@11465 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 12 insertions, 3 deletions

diff --git a/chrome/browser/download/save_package.cc b/chrome/browser/download/save_package.cc
index 12d2443..158432d 100644
--- a/chrome/browser/download/save_package.cc
+++ b/chrome/browser/download/save_package.cc
@@ -963,9 +963,18 @@ FilePath SavePackage::GetSuggestNameForSaveAs(PrefService* prefs,
   // If not, initialize it with default directory.
   if (!prefs->IsPrefRegistered(prefs::kSaveFileDefaultDirectory)) {
     FilePath default_save_path;
-    if (!PathService::Get(chrome::DIR_DEFAULT_DOWNLOADS,
-                          &default_save_path))
-      NOTREACHED();
+    if (!prefs->IsPrefRegistered(prefs::kDownloadDefaultDirectory)) {
+      if (!PathService::Get(chrome::DIR_DEFAULT_DOWNLOADS,
+                            &default_save_path)) {
+        NOTREACHED();
+      }
+    } else {
+      StringPrefMember default_download_path;
+      default_download_path.Init(prefs::kDownloadDefaultDirectory,
+                                 prefs, NULL);
+      default_save_path = FilePath::FromWStringHack(
+                          default_download_path.GetValue());
+    }
     prefs->RegisterFilePathPref(prefs::kSaveFileDefaultDirectory,
                                 default_save_path);
   }