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

Unified Diff: chrome/test/live_sync/multiple_client_bookmarks_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_bookmarks_sync_test.cc
diff --git a/chrome/test/live_sync/multiple_client_bookmarks_sync_test.cc b/chrome/test/live_sync/multiple_client_bookmarks_sync_test.cc
index c02ef4af61ea0a9764604ee2f35e24dc4bce2cbf..ddc4abdbaeaf2f38cf1277fd7a047304767c2690 100644
--- a/chrome/test/live_sync/multiple_client_bookmarks_sync_test.cc
+++ b/chrome/test/live_sync/multiple_client_bookmarks_sync_test.cc
@@ -7,6 +7,9 @@
#include "chrome/test/live_sync/bookmarks_helper.h"
#include "chrome/test/live_sync/live_sync_test.h"
+using bookmarks_helper::AddURL;
+using bookmarks_helper::AllModelsMatch;
+
class MultipleClientBookmarksSyncTest : public LiveSyncTest {
public:
MultipleClientBookmarksSyncTest() : LiveSyncTest(MULTIPLE_CLIENT) {}
@@ -20,10 +23,9 @@ IN_PROC_BROWSER_TEST_F(MultipleClientBookmarksSyncTest, Sanity) {
ASSERT_TRUE(SetupSync()) << "SetupSync() failed.";
DisableVerifier();
for (int i = 0; i < num_clients(); ++i) {
- ASSERT_TRUE(BookmarksHelper::AddURL(
- i, base::StringPrintf(L"Google URL %d", i),
- GURL(StringPrintf("http://www.google.com/%d", i))) != NULL);
+ ASSERT_TRUE(AddURL(i, base::StringPrintf(L"Google URL %d", i),
+ GURL(StringPrintf("http://www.google.com/%d", i))) != NULL);
}
ASSERT_TRUE(AwaitQuiescence());
- ASSERT_TRUE(BookmarksHelper::AllModelsMatch());
+ ASSERT_TRUE(AllModelsMatch());
}
« no previous file with comments | « chrome/test/live_sync/migration_errors_test.cc ('k') | chrome/test/live_sync/multiple_client_live_typed_urls_sync_test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698