Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(785)

Unified Diff: chrome/browser/download/download_danger_prompt_browsertest.cc

Issue 16924017: A few minor changes to the chrome.downloads extension API (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: @r208716 Created 7 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/download/download_danger_prompt_browsertest.cc
diff --git a/chrome/browser/download/download_danger_prompt_browsertest.cc b/chrome/browser/download/download_danger_prompt_browsertest.cc
index 5a7c9e8b9b5e45a5c4903c5b262421d062f3b71f..a32133ff7fb6791ac93092378fb55892661120d2 100644
--- a/chrome/browser/download/download_danger_prompt_browsertest.cc
+++ b/chrome/browser/download/download_danger_prompt_browsertest.cc
@@ -89,7 +89,8 @@ class DownloadDangerPromptTest : public InProcessBrowserTest {
base::Bind(&DownloadDangerPromptTest::PromptCallback, this,
DownloadDangerPrompt::ACCEPT),
base::Bind(&DownloadDangerPromptTest::PromptCallback, this,
- DownloadDangerPrompt::CANCEL));
+ DownloadDangerPrompt::CANCEL),
+ base::Bind(&DownloadDangerPromptTest::PromptDestroyed, this));
content::RunAllPendingInMessageLoop();
}
@@ -97,6 +98,9 @@ class DownloadDangerPromptTest : public InProcessBrowserTest {
EXPECT_FALSE(did_receive_callback_);
EXPECT_EQ(expected_action_, action);
did_receive_callback_ = true;
+ }
+
+ void PromptDestroyed() {
prompt_ = NULL;
}

Powered by Google App Engine
This is Rietveld 408576698