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

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

Issue 7461109: Allow sync integration tests to operate on multiple datatypes: Typed Urls (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: CR feedback; rebase Created 9 years, 4 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
Index: chrome/test/live_sync/single_client_preferences_sync_test.cc
diff --git a/chrome/test/live_sync/single_client_preferences_sync_test.cc b/chrome/test/live_sync/single_client_preferences_sync_test.cc
index 25db722630d9e14e035717c1e6c88fea2e1133b9..1618ddaebefa536f8497dee1dac2ce39383b673a 100644
--- a/chrome/test/live_sync/single_client_preferences_sync_test.cc
+++ b/chrome/test/live_sync/single_client_preferences_sync_test.cc
@@ -7,6 +7,9 @@
#include "chrome/test/live_sync/preferences_helper.h"
#include "chrome/test/live_sync/live_sync_test.h"
+using preferences_helper::BooleanPrefMatches;
+using preferences_helper::ChangeBooleanPref;
+
class SingleClientPreferencesSyncTest : public LiveSyncTest {
public:
SingleClientPreferencesSyncTest() : LiveSyncTest(SINGLE_CLIENT) {}
@@ -18,11 +21,9 @@ class SingleClientPreferencesSyncTest : public LiveSyncTest {
IN_PROC_BROWSER_TEST_F(SingleClientPreferencesSyncTest, Sanity) {
ASSERT_TRUE(SetupSync()) << "SetupSync() failed.";
- ASSERT_TRUE(PreferencesHelper::BooleanPrefMatches(
- prefs::kHomePageIsNewTabPage));
- PreferencesHelper::ChangeBooleanPref(0, prefs::kHomePageIsNewTabPage);
+ ASSERT_TRUE(BooleanPrefMatches(prefs::kHomePageIsNewTabPage));
+ ChangeBooleanPref(0, prefs::kHomePageIsNewTabPage);
ASSERT_TRUE(GetClient(0)->AwaitSyncCycleCompletion(
"Waiting for prefs change."));
- ASSERT_TRUE(PreferencesHelper::BooleanPrefMatches(
- prefs::kHomePageIsNewTabPage));
+ ASSERT_TRUE(BooleanPrefMatches(prefs::kHomePageIsNewTabPage));
}

Powered by Google App Engine
This is Rietveld 408576698