Revert 155351 - Make DownloadManager::GetAllDownloads return all downloads regardless of state, per…

content/browser/download/download_manager_impl.cc

Index: content/browser/download/download_manager_impl.cc
===================================================================
--- content/browser/download/download_manager_impl.cc	(revision 155501)
+++ content/browser/download/download_manager_impl.cc	(working copy)
@@ -349,6 +349,22 @@
   }
 }
 
+void DownloadManagerImpl::GetAllDownloads(
+    const FilePath& dir_path, DownloadVector* result) {
+  DCHECK(result);
+
+  for (DownloadMap::iterator it = downloads_.begin();
+       it != downloads_.end(); ++it) {
+    DownloadItemImpl* item = it->second;
+    // TODO(benjhayden): Don't check IsPersisted().
+    if (!item->IsTemporary() &&
+        item->IsPersisted() &&
+        (dir_path.empty() ||
+         item->GetTargetFilePath().DirName() == dir_path))
+      result->push_back(item);
+  }
+}
+
 void DownloadManagerImpl::SearchDownloads(const string16& query,
                                           DownloadVector* result) {
   string16 query_lower(base::i18n::ToLower(query));
@@ -966,13 +982,6 @@
   return ContainsKey(downloads_, download_id) ? downloads_[download_id] : NULL;
 }
 
-void DownloadManagerImpl::GetAllDownloads(DownloadVector* downloads) {
-  for (DownloadMap::iterator it = downloads_.begin();
-       it != downloads_.end(); ++it) {
-    downloads->push_back(it->second);
-  }
-}
-
 DownloadItem* DownloadManagerImpl::GetActiveDownloadItem(int download_id) {
   if (ContainsKey(active_downloads_, download_id))
     return active_downloads_[download_id];