Rearrange Should*Download delegate calls to be more useful and intuitive.

content/browser/download/download_item.cc

Index: content/browser/download/download_item.cc
diff --git a/content/browser/download/download_item.cc b/content/browser/download/download_item.cc
index 249a1afa127aa8bd6a0c9bda595f5bafc78f2785..a8ed7beb003da3ea3848298773b347b83995d447 100644
--- a/content/browser/download/download_item.cc
+++ b/content/browser/download/download_item.cc
@@ -296,8 +296,7 @@ void DownloadItem::OpenDownload() {
   if (!open_enabled_)
     return;
 
-  if (download_manager_->delegate()->ShouldOpenDownload(this))
-    content::GetContentClient()->browser()->OpenItem(full_path());
+  content::GetContentClient()->browser()->OpenItem(full_path());
 }
 
 void DownloadItem::ShowDownloadInShell() {
@@ -397,7 +396,7 @@ void DownloadItem::MarkAsComplete() {
   TransitionTo(COMPLETE);
 }
 
-void DownloadItem::CompleteDelayedDownload() {
+void DownloadItem::DelayedDownloadOpened() {
   auto_opened_ = true;
   Completed();
 }
@@ -618,7 +617,7 @@ void DownloadItem::OnDownloadRenamedToFinalName(const FilePath& full_path) {
 
   Rename(full_path);
 
-  if (download_manager_->delegate()->ShouldCompleteDownload(this)) {
+  if (download_manager_->delegate()->ShouldOpenDownload(this)) {
     Completed();
   } else {
     delegate_delayed_complete_ = true;