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

Unified Diff: chrome/browser/translate/translate_manager_browsertest.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/translate/translate_manager_browsertest.cc
diff --git a/chrome/browser/translate/translate_manager_browsertest.cc b/chrome/browser/translate/translate_manager_browsertest.cc
index e5d26d99d20c1eee837f5e97142780f983490e67..1aa579b4c114fe2ff58b3c85551875ca328fc2ca 100644
--- a/chrome/browser/translate/translate_manager_browsertest.cc
+++ b/chrome/browser/translate/translate_manager_browsertest.cc
@@ -205,7 +205,7 @@ class TranslateManagerTest : public ChromeRenderViewHostTestHarness,
// The TranslateManager class processes the navigation entry committed
// notification in a posted task; process that task.
- MessageLoop::current()->RunAllPending();
+ MessageLoop::current()->RunUntilIdle();
}
virtual void Observe(int type,
@@ -813,7 +813,7 @@ TEST_F(TranslateManagerTest, ReloadFromLocationBar) {
// The TranslateManager class processes the navigation entry committed
// notification in a posted task; process that task.
- MessageLoop::current()->RunAllPending();
+ MessageLoop::current()->RunUntilIdle();
EXPECT_TRUE(GetTranslateInfoBar() != NULL);
}
@@ -1454,7 +1454,7 @@ TEST_F(TranslateManagerTest, ScriptExpires) {
0, 0, "fr", "en", TranslateErrors::NONE));
// A task should have been posted to clear the script, run it.
- MessageLoop::current()->RunAllPending();
+ MessageLoop::current()->RunUntilIdle();
// Do another navigation and translation.
SimulateNavigation(GURL("http://www.google.es"), "es", true);
« no previous file with comments | « chrome/browser/themes/theme_syncable_service_unittest.cc ('k') | chrome/browser/user_style_sheet_watcher_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698