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

Unified Diff: chrome/test/live_sync/two_client_live_preferences_sync_test.cc

Issue 3037019: Fixing broken Integration tests. (Closed)
Patch Set: Adding periods after comments. Created 10 years, 5 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
« no previous file with comments | « chrome/test/live_sync/two_client_live_bookmarks_sync_test.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/test/live_sync/two_client_live_preferences_sync_test.cc
diff --git a/chrome/test/live_sync/two_client_live_preferences_sync_test.cc b/chrome/test/live_sync/two_client_live_preferences_sync_test.cc
index 390806b0cfd4e901607f9c2c9ecca1fdda1503ae..5e72c13988c8a7f4cd4eaf3b1b804182676758fa 100644
--- a/chrome/test/live_sync/two_client_live_preferences_sync_test.cc
+++ b/chrome/test/live_sync/two_client_live_preferences_sync_test.cc
@@ -21,12 +21,12 @@ IN_PROC_BROWSER_TEST_F(TwoClientLivePreferencesSyncTest, Sanity) {
GetPrefs(1)->GetBoolean(prefs::kHomePageIsNewTabPage));
}
-IN_PROC_BROWSER_TEST_F(TwoClientLivePreferencesSyncTest, FAILS_Race) {
+IN_PROC_BROWSER_TEST_F(TwoClientLivePreferencesSyncTest, Race) {
ASSERT_TRUE(SetupSync()) << "SetupSync() failed.";
GetPrefs(0)->SetString(prefs::kHomePage, "http://www.google.com/1");
GetPrefs(1)->SetString(prefs::kHomePage, "http://www.google.com/2");
- EXPECT_TRUE(GetClient(1)->AwaitMutualSyncCycleCompletion(GetClient(0)));
+ ASSERT_TRUE(ProfileSyncServiceTestHarness::AwaitQuiescence(clients()));
EXPECT_EQ(GetPrefs(0)->GetString(prefs::kHomePage),
GetPrefs(1)->GetString(prefs::kHomePage));
« no previous file with comments | « chrome/test/live_sync/two_client_live_bookmarks_sync_test.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698