Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1501)

Unified Diff: chrome/browser/download/download_manager.h

Issue 7324031: Revert 91861 - When the download folder does not exist, change the download folder to a user's "D... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 9 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/download/download_file_manager.cc ('k') | chrome/browser/download/download_manager.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/download/download_manager.h
===================================================================
--- chrome/browser/download/download_manager.h (revision 91868)
+++ chrome/browser/download/download_manager.h (working copy)
@@ -87,9 +87,7 @@
// Called immediately after the DownloadManager puts up a select file
// dialog.
// |id| indicates which download opened the dialog.
- // |suggested_path| indicates the path suggested in the dialog.
- virtual void SelectFileDialogDisplayed(
- int32 id, const FilePath& suggested_path) {}
+ virtual void SelectFileDialogDisplayed(int32 id) {}
protected:
virtual ~Observer() {}
@@ -313,7 +311,7 @@
// stalls from interacting with the file system.
void CheckIfSuggestedPathExists(int32 download_id,
DownloadStateInfo state,
- const FilePath& download_save_dir);
+ const FilePath& default_path);
// Called on the UI thread once the DownloadManager has determined whether the
// suggested file path exists.
« no previous file with comments | « chrome/browser/download/download_file_manager.cc ('k') | chrome/browser/download/download_manager.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698