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

Unified Diff: chrome/browser/safe_browsing/download_protection_service_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/safe_browsing/download_protection_service_unittest.cc
diff --git a/chrome/browser/safe_browsing/download_protection_service_unittest.cc b/chrome/browser/safe_browsing/download_protection_service_unittest.cc
index fde46f19ad3372ff03e8418232c3a913eedc1ef4..7a2b01f6e1f38eaebc40831cdeac2d3cf5de2a59 100644
--- a/chrome/browser/safe_browsing/download_protection_service_unittest.cc
+++ b/chrome/browser/safe_browsing/download_protection_service_unittest.cc
@@ -127,7 +127,7 @@ class DownloadProtectionServiceTest : public testing::Test {
download_service_ = sb_service_->download_protection_service();
download_service_->signature_util_ = signature_util_;
download_service_->SetEnabled(true);
- msg_loop_.RunAllPending();
+ msg_loop_.RunUntilIdle();
FilePath source_path;
ASSERT_TRUE(PathService::Get(base::DIR_SOURCE_ROOT, &source_path));
@@ -182,7 +182,7 @@ class DownloadProtectionServiceTest : public testing::Test {
void FlushThreadMessageLoops() {
BrowserThread::GetBlockingPool()->FlushForTesting();
FlushMessageLoop(BrowserThread::IO);
- msg_loop_.RunAllPending();
+ msg_loop_.RunUntilIdle();
}
// Proxy for private method.
@@ -214,7 +214,7 @@ class DownloadProtectionServiceTest : public testing::Test {
private:
// Helper functions for FlushThreadMessageLoops.
void RunAllPendingAndQuitUI() {
- MessageLoop::current()->RunAllPending();
+ MessageLoop::current()->RunUntilIdle();
BrowserThread::PostTask(
BrowserThread::UI,
FROM_HERE,

Powered by Google App Engine
This is Rietveld 408576698