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 5b0319afd2ef8447ca6af4fb5ab80aaac2393c21..a4731f31366c9a140ddce091a24afae3a860d547 100644
--- a/content/browser/download/download_manager_impl.cc
+++ b/content/browser/download/download_manager_impl.cc
@@ -973,21 +973,10 @@ 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;
 }
 
-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 {