Remove DownloadManager::GetDownloadItem in favor of GetDownload()

content/browser/download/download_manager_impl.cc

Index: content/browser/download/download_manager_impl.cc
diff --git a/content/browser/download/download_manager_impl.cc b/content/browser/download/download_manager_impl.cc
index 8b2e2dbd901ff29385aac766434bf7bc85ce7b2e..bfd5ac0be555b9f4c8c8360750e5d13c4c95b338 100644
--- a/content/browser/download/download_manager_impl.cc
+++ b/content/browser/download/download_manager_impl.cc
@@ -927,11 +927,6 @@ void DownloadManagerImpl::NotifyModelChanged() {
   FOR_EACH_OBSERVER(Observer, observers_, ModelChanged(this));
 }
 
-DownloadItem* DownloadManagerImpl::GetDownloadItem(int download_id) {
-  DownloadItem* download = GetDownload(download_id);
-  return (download && download->IsPersisted()) ? download : NULL;
-}
-
 DownloadItem* DownloadManagerImpl::GetDownload(int download_id) {
   return ContainsKey(downloads_, download_id) ? downloads_[download_id] : NULL;
 }
@@ -943,12 +938,6 @@ void DownloadManagerImpl::GetAllDownloads(DownloadVector* downloads) {
   }
 }
 
-DownloadItem* DownloadManagerImpl::GetActiveDownloadItem(int download_id) {
-  if (ContainsKey(active_downloads_, download_id))
-    return active_downloads_[download_id];
-  return NULL;
-}
-
 // Confirm that everything in all maps is also in |downloads_|, and that
 // everything in |downloads_| is also in some other map.
 void DownloadManagerImpl::AssertContainersConsistent() const {