Remove DownloadManager::GetDownloadItem in favor of GetDownload()

chrome/browser/plugins/plugin_installer.cc

Index: chrome/browser/plugins/plugin_installer.cc
diff --git a/chrome/browser/plugins/plugin_installer.cc b/chrome/browser/plugins/plugin_installer.cc
index 79e06813df9a7411fd102b3589402ba736c2d91f..32ac455b5ff84f74f5e2848dfd8a377c8c7c9c28 100644
--- a/chrome/browser/plugins/plugin_installer.cc
+++ b/chrome/browser/plugins/plugin_installer.cc
@@ -230,8 +230,13 @@ void PluginInstaller::DownloadStarted(
     DownloadError(msg);
     return;
   }
-  DownloadItem* download_item =
-      dlm->GetActiveDownloadItem(download_id.local());
+  DownloadItem* download_item = dlm->GetDownload(download_id.local());
+  // TODO(benjhayden): DCHECK(item && item->IsInProgress()) after figuring out
+  // why DownloadStarted may get net:OK but an invalid id.
+  if (!download_item) {
+    DownloadError("Download not found");
+    return;
+  }
   download_item->SetOpenWhenComplete(true);
   download_item->AddObserver(this);
 }