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

Unified Diff: chrome/test/live_sync/multiple_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/multiple_client_preferences_sync_test.cc
diff --git a/chrome/test/live_sync/multiple_client_preferences_sync_test.cc b/chrome/test/live_sync/multiple_client_preferences_sync_test.cc
index 555c888b748ab61c07920ee6c02b0cb1f3391dac..d0534ddb9be8edc7c61019f0c5b53e678e39276e 100644
--- a/chrome/test/live_sync/multiple_client_preferences_sync_test.cc
+++ b/chrome/test/live_sync/multiple_client_preferences_sync_test.cc
@@ -8,6 +8,9 @@
#include "chrome/test/live_sync/live_sync_test.h"
#include "chrome/test/live_sync/preferences_helper.h"
+using preferences_helper::ChangeListPref;
+using preferences_helper::ListPrefMatches;
+
class MultipleClientPreferencesSyncTest : public LiveSyncTest {
public:
MultipleClientPreferencesSyncTest() : LiveSyncTest(MULTIPLE_CLIENT) {}
@@ -25,10 +28,9 @@ IN_PROC_BROWSER_TEST_F(MultipleClientPreferencesSyncTest, Sanity) {
ListValue urls;
urls.Append(Value::CreateStringValue(
base::StringPrintf("http://www.google.com/%d", i)));
- PreferencesHelper::ChangeListPref(i, prefs::kURLsToRestoreOnStartup, urls);
+ ChangeListPref(i, prefs::kURLsToRestoreOnStartup, urls);
}
ASSERT_TRUE(AwaitQuiescence());
- ASSERT_TRUE(PreferencesHelper::ListPrefMatches(
- prefs::kURLsToRestoreOnStartup));
+ ASSERT_TRUE(ListPrefMatches(prefs::kURLsToRestoreOnStartup));
}

Powered by Google App Engine
This is Rietveld 408576698