Chromium Code Reviews| Index: chrome/browser/download/download_manager.cc |
| diff --git a/chrome/browser/download/download_manager.cc b/chrome/browser/download/download_manager.cc |
| index 58090f2d492e18f00c5dc16dca08589f9e43e6e1..777624504231bab28f725ced2a273c7085a4f947 100644 |
| --- a/chrome/browser/download/download_manager.cc |
| +++ b/chrome/browser/download/download_manager.cc |
| @@ -368,11 +368,6 @@ void DownloadManager::CheckIfSuggestedPathExists(int32 download_id, |
| const FilePath& default_path) { |
| DCHECK(BrowserThread::CurrentlyOn(BrowserThread::FILE)); |
| - // Make sure the default download directory exists. |
| - // TODO(phajdan.jr): only create the directory when we're sure the user |
| - // is going to save there and not to another directory of his choice. |
| - file_util::CreateDirectory(default_path); |
|
Paweł Hajdan Jr.
2011/05/30 14:15:49
Please also add a test for a case where we're down
haraken1
2011/05/31 12:41:31
I added DownloadTest.DownloadedDirectory.
|
| - |
| // Check writability of the suggested path. If we can't write to it, default |
| // to the user's "My Documents" directory. We'll prompt them in this case. |
| FilePath dir = state.suggested_path.DirName(); |