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

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

Issue 11413050: chrome/browser: Update calls from RunAllPending() to RunUntilIdle(). (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 1 month 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/chrome_download_manager_delegate_unittest.cc
diff --git a/chrome/browser/download/chrome_download_manager_delegate_unittest.cc b/chrome/browser/download/chrome_download_manager_delegate_unittest.cc
index a3b6506dae8c72abec2ef8f444a6d469549ffe62..fc32f552953976702c674087a068506466265303 100644
--- a/chrome/browser/download/chrome_download_manager_delegate_unittest.cc
+++ b/chrome/browser/download/chrome_download_manager_delegate_unittest.cc
@@ -312,7 +312,7 @@ void ChromeDownloadManagerDelegateTest::SetUp() {
}
void ChromeDownloadManagerDelegateTest::TearDown() {
- message_loop_.RunAllPending();
+ message_loop_.RunUntilIdle();
delegate_->Shutdown();
ChromeRenderViewHostTestHarness::TearDown();
}
@@ -447,7 +447,7 @@ void ChromeDownloadManagerDelegateTest::RunTestCaseWithDownloadItem(
item,
base::Bind(&ChromeDownloadManagerDelegateTest::DownloadTargetVerifier,
factory.GetWeakPtr(), base::Unretained(&test_case))));
- message_loop_.RunAllPending();
+ message_loop_.RunUntilIdle();
VerifyAndClearExpectations();
}

Powered by Google App Engine
This is Rietveld 408576698