Remove DownloadManager::GetDownloadItem in favor of GetDownload()

content/shell/shell_download_manager_delegate.cc

Index: content/shell/shell_download_manager_delegate.cc
diff --git a/content/shell/shell_download_manager_delegate.cc b/content/shell/shell_download_manager_delegate.cc
index 5dfb4921678ec9c88ea2f8089ab59b2293cf8903..0f7777b5112f73aae527ceaeade9f7a054ed3dd3 100644
--- a/content/shell/shell_download_manager_delegate.cc
+++ b/content/shell/shell_download_manager_delegate.cc
@@ -123,8 +123,7 @@ void ShellDownloadManagerDelegate::ChooseDownloadPath(
     const DownloadTargetCallback& callback,
     const FilePath& suggested_path) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
-  DownloadItem* item =
-      download_manager_->GetActiveDownloadItem(download_id);
+  DownloadItem* item = download_manager_->GetDownload(download_id);

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

Same comment as in CDMD: this seems like a noticeable behavior change--can you
make a pitch as to why we'd want it?

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

Done.

   if (!item)
     return;