Reland DownloadManager::GetAllDownloads actually does now

content/public/browser/download_manager.h

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..917780468d192c7df96f2c41888c297719b379c6 100644
--- a/content/public/browser/download_manager.h
+++ b/content/public/browser/download_manager.h
@@ -104,17 +104,9 @@ class CONTENT_EXPORT DownloadManager
 
   typedef std::vector<DownloadItem*> DownloadVector;
 
-  // If |dir_path| is empty, appends all temporary downloads to |*result|.
-  // Otherwise, appends all temporary downloads that reside in |dir_path| to
-  // |*result|.
-  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.