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

Side by Side Diff: chrome/browser/download/download_manager.h

Issue 6973052: When the download folder does not exist, change the download folder to a user's "Downloads" (Closed) Base URL: http://git.chromium.org/git/chromium.git@trunk
Patch Set: Correct typo Created 9 years, 6 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 // 4 //
5 // The DownloadManager object manages the process of downloading, including 5 // The DownloadManager object manages the process of downloading, including
6 // updates to the history system and providing the information for displaying 6 // updates to the history system and providing the information for displaying
7 // the downloads view in the Destinations tab. There is one DownloadManager per 7 // the downloads view in the Destinations tab. There is one DownloadManager per
8 // active profile in Chrome. 8 // active profile in Chrome.
9 // 9 //
10 // Download observers: 10 // Download observers:
(...skipping 68 matching lines...) Expand 10 before | Expand all | Expand 10 after
79 // of downloads. 79 // of downloads.
80 virtual void ModelChanged() = 0; 80 virtual void ModelChanged() = 0;
81 81
82 // Called when the DownloadManager is being destroyed to prevent Observers 82 // Called when the DownloadManager is being destroyed to prevent Observers
83 // from calling back to a stale pointer. 83 // from calling back to a stale pointer.
84 virtual void ManagerGoingDown() {} 84 virtual void ManagerGoingDown() {}
85 85
86 // Called immediately after the DownloadManager puts up a select file 86 // Called immediately after the DownloadManager puts up a select file
87 // dialog. 87 // dialog.
88 // |id| indicates which download opened the dialog. 88 // |id| indicates which download opened the dialog.
89 virtual void SelectFileDialogDisplayed(int32 id) {} 89 virtual void SelectFileDialogDisplayed(int32 id, FilePath& path) {}
Paweł Hajdan Jr. 2011/05/31 17:15:53 Make sure to explain the meaning of |path|. It see
haraken1 2011/06/02 09:13:22 Done.
90 90
91 protected: 91 protected:
92 virtual ~Observer() {} 92 virtual ~Observer() {}
93 }; 93 };
94 94
95 // Return all temporary downloads that reside in the specified directory. 95 // Return all temporary downloads that reside in the specified directory.
96 void GetTemporaryDownloads(const FilePath& dir_path, 96 void GetTemporaryDownloads(const FilePath& dir_path,
97 std::vector<DownloadItem*>* result); 97 std::vector<DownloadItem*>* result);
98 98
99 // Return all non-temporary downloads in the specified directory that are 99 // Return all non-temporary downloads in the specified directory that are
(...skipping 311 matching lines...) Expand 10 before | Expand all | Expand 10 after
411 // The "Save As" dialog box used to ask the user where a file should be 411 // The "Save As" dialog box used to ask the user where a file should be
412 // saved. 412 // saved.
413 scoped_refptr<SelectFileDialog> select_file_dialog_; 413 scoped_refptr<SelectFileDialog> select_file_dialog_;
414 414
415 scoped_ptr<OtherDownloadManagerObserver> other_download_manager_observer_; 415 scoped_ptr<OtherDownloadManagerObserver> other_download_manager_observer_;
416 416
417 DISALLOW_COPY_AND_ASSIGN(DownloadManager); 417 DISALLOW_COPY_AND_ASSIGN(DownloadManager);
418 }; 418 };
419 419
420 #endif // CHROME_BROWSER_DOWNLOAD_DOWNLOAD_MANAGER_H_ 420 #endif // CHROME_BROWSER_DOWNLOAD_DOWNLOAD_MANAGER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698