[Resumption 10/12] Use DI::IsDone to check for terminal downloads.

chrome/browser/download/download_danger_prompt.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/download/download_danger_prompt.h"
 
 #include "base/bind.h"
 #include "chrome/browser/download/chrome_download_manager_delegate.h"
 #include "chrome/browser/ui/tab_modal_confirm_dialog.h"
 #include "chrome/browser/ui/tab_modal_confirm_dialog_delegate.h"
@@ skipping 76 matching lines @@
 void DownloadDangerPromptImpl::InvokeActionForTesting(Action action) {
   if (action == ACCEPT)
     Accept();
   else
     Cancel();
 }
 
 void DownloadDangerPromptImpl::OnDownloadUpdated(
     content::DownloadItem* download) {
   // If the download is nolonger dangerous (accepted externally) or the download
-  // doesn't exist anymore, the download danger prompt is no longer necessary.
-  if ((download->GetState() != content::DownloadItem::IN_PROGRESS)
-      || !download->IsDangerous())
+  // is in a terminal state, then the download danger prompt is no longer
+  // necessary.
+  if (!download->IsDangerous() || download->IsDone())
     Cancel();
 }
 
 void DownloadDangerPromptImpl::OnDownloadOpened(
     content::DownloadItem* download) {
 }
 
 string16 DownloadDangerPromptImpl::GetTitle() {
   return l10n_util::GetStringUTF16(IDS_CONFIRM_KEEP_DANGEROUS_DOWNLOAD_TITLE);
 }
@@ skipping 63 matching lines @@
     content::WebContents* web_contents,
     bool show_context,
     const base::Closure& accepted,
     const base::Closure& canceled) {
   DownloadDangerPromptImpl* prompt = new DownloadDangerPromptImpl(
       item, web_contents, show_context, accepted, canceled);
   // |prompt| will be deleted when the dialog is done.
   TabModalConfirmDialog::Create(prompt, web_contents);
   return prompt;
 }