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

Side by Side Diff: chrome/test/live_sync/many_client_live_preferences_sync_test.cc

Issue 6462015: Cleanup more test code. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: wtf windows is this me Created 9 years, 10 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2010 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/sync/profile_sync_service_harness.h"
5 #include "chrome/common/pref_names.h" 6 #include "chrome/common/pref_names.h"
6 #include "chrome/test/live_sync/live_preferences_sync_test.h" 7 #include "chrome/test/live_sync/live_preferences_sync_test.h"
7 8
8 // TODO(rsimha): Enable once http://crbug.com/69604 is fixed. 9 // TODO(rsimha): Enable once http://crbug.com/69604 is fixed.
9 IN_PROC_BROWSER_TEST_F(ManyClientLivePreferencesSyncTest, DISABLED_Sanity) { 10 IN_PROC_BROWSER_TEST_F(ManyClientLivePreferencesSyncTest, DISABLED_Sanity) {
10 ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; 11 ASSERT_TRUE(SetupSync()) << "SetupSync() failed.";
11 12
12 bool new_value = !GetVerifierPrefs()->GetBoolean( 13 bool new_value = !GetVerifierPrefs()->GetBoolean(
13 prefs::kHomePageIsNewTabPage); 14 prefs::kHomePageIsNewTabPage);
14 GetVerifierPrefs()->SetBoolean(prefs::kHomePageIsNewTabPage, new_value); 15 GetVerifierPrefs()->SetBoolean(prefs::kHomePageIsNewTabPage, new_value);
15 GetPrefs(0)->SetBoolean(prefs::kHomePageIsNewTabPage, new_value); 16 GetPrefs(0)->SetBoolean(prefs::kHomePageIsNewTabPage, new_value);
16 17
17 ASSERT_TRUE(GetClient(0)->AwaitGroupSyncCycleCompletion(clients())); 18 ASSERT_TRUE(GetClient(0)->AwaitGroupSyncCycleCompletion(clients()));
18 19
19 for (int i = 0; i < num_clients(); ++i) { 20 for (int i = 0; i < num_clients(); ++i) {
20 ASSERT_EQ(GetVerifierPrefs()->GetBoolean(prefs::kHomePageIsNewTabPage), 21 ASSERT_EQ(GetVerifierPrefs()->GetBoolean(prefs::kHomePageIsNewTabPage),
21 GetPrefs(i)->GetBoolean(prefs::kHomePageIsNewTabPage)); 22 GetPrefs(i)->GetBoolean(prefs::kHomePageIsNewTabPage));
22 } 23 }
23 } 24 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698