Remove DownloadManager::GetDownloadItem in favor of GetDownload()

chrome/browser/ui/webui/downloads_dom_handler.cc

Index: chrome/browser/ui/webui/downloads_dom_handler.cc
diff --git a/chrome/browser/ui/webui/downloads_dom_handler.cc b/chrome/browser/ui/webui/downloads_dom_handler.cc
index be92a11a353bae762cca5328e4300147b70a4951..15c8417793e96a99b604e4e272d26a0a5f68f2b2 100644
--- a/chrome/browser/ui/webui/downloads_dom_handler.cc
+++ b/chrome/browser/ui/webui/downloads_dom_handler.cc
@@ -500,8 +500,7 @@ void DownloadsDOMHandler::ShowDangerPrompt(
 }
 
 void DownloadsDOMHandler::DangerPromptAccepted(int download_id) {
-  content::DownloadItem* item = download_manager_->GetActiveDownloadItem(
-      download_id);
+  content::DownloadItem* item = download_manager_->GetDownload(download_id);

[Randy Smith (Not in Mondays), 2012/09/11 19:02:21]

I think we want a cancellation/interrupt guard somewhere along this pathway.  I
don't care if it's here or in DangerousDownloadValidated().

[benjhayden, 2012/09/12 18:17:25]

Done.

   if (!item)
     return;
   CountDownloadsDOMEvents(DOWNLOADS_DOM_EVENT_SAVE_DANGEROUS);