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

Unified Diff: content/public/browser/download_manager.h

Issue 10913015: Reland DownloadManager::GetAllDownloads actually does now (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: . Created 8 years, 3 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
Index: content/public/browser/download_manager.h
diff --git a/content/public/browser/download_manager.h b/content/public/browser/download_manager.h
index 7501f0ef1dea5b153d8e47d642b4328687d1708c..056a29def38eee03e5b1890f79ac7c1972254e90 100644
--- a/content/public/browser/download_manager.h
+++ b/content/public/browser/download_manager.h
@@ -110,11 +110,9 @@ class CONTENT_EXPORT DownloadManager
virtual void GetTemporaryDownloads(const FilePath& dir_path,
DownloadVector* result) = 0;
- // If |dir_path| is empty, appends all non-temporary downloads to |*result|.
- // Otherwise, appends all non-temporary downloads that reside in |dir_path|
- // to |*result|.
- virtual void GetAllDownloads(const FilePath& dir_path,
- DownloadVector* result) = 0;
+ // Add all download items to |downloads|, no matter the type or state, without
+ // clearing |downloads| first.
+ virtual void GetAllDownloads(DownloadVector* downloads) = 0;
// Returns all non-temporary downloads matching |query|. Empty query matches
// everything.

Powered by Google App Engine
This is Rietveld 408576698