A few minor changes to the chrome.downloads extension API

chrome/browser/download/download_danger_prompt.h

 // 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.
 
 #ifndef CHROME_BROWSER_DOWNLOAD_DOWNLOAD_DANGER_PROMPT_H_
 #define CHROME_BROWSER_DOWNLOAD_DOWNLOAD_DANGER_PROMPT_H_
 
 #include "base/callback_forward.h"
 
 namespace content {
@@ skipping 22 matching lines @@
   // progress, or if the tab corresponding to |web_contents| is
   // closing. The returned DownloadDangerPrompt* is only used for testing. The
   // caller does not own the object and receive no guarantees about lifetime.
   // If |show_context|, then the prompt message will contain some information
   // about the download and its danger; otherwise it won't.
   static DownloadDangerPrompt* Create(
       content::DownloadItem* item,
       content::WebContents* web_contents,
       bool show_context,
       const base::Closure& accepted,
-      const base::Closure& canceled);
+      const base::Closure& canceled,
+      const base::Closure& destroyed);

[asanka, 2013/06/28 21:59:59]

I'm guessing this would be the equivalent of the new Dismissed action being
added in https://codereview.chromium.org/18179004/? Could we add a callback that
takes a result parameter instead of adding closures?

[benjhayden, 2013/07/11 15:32:28]

Done.

 
  protected:
   friend class DownloadDangerPromptTest;
 
   // Only to be used by tests. Subclasses must override to manually call the
   // respective button click handler.
   virtual void InvokeActionForTesting(Action action) = 0;
 };
 
 #endif  // CHROME_BROWSER_DOWNLOAD_DOWNLOAD_DANGER_PROMPT_H_