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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: chrome/browser/sync/test/integration/two_client_bookmarks_sync_test.cc

Issue 2494633005: Revert of [sync] "TwoClientBookmarksSyncTest.Sanity" is flaky (Closed)
Patch Set: Created 4 years, 1 month 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/browser/sync/test/integration/sync_test.cc ('k') | components/sync/driver/glue/sync_backend_host_impl.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/sync/test/integration/two_client_bookmarks_sync_test.cc
diff --git a/chrome/browser/sync/test/integration/two_client_bookmarks_sync_test.cc b/chrome/browser/sync/test/integration/two_client_bookmarks_sync_test.cc
index f04b7a1bb29fb322a9cc9c79a4188e64678a96ca..6db9fdaa6906bface38f813ca5e0013e775cbf03 100644
--- a/chrome/browser/sync/test/integration/two_client_bookmarks_sync_test.cc
+++ b/chrome/browser/sync/test/integration/two_client_bookmarks_sync_test.cc
@@ -109,26 +109,30 @@
ASSERT_NE(nullptr, AddURL(0, "Google", google_url));
ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
ASSERT_NE(nullptr, AddURL(1, "Yahoo", GURL("http://www.yahoo.com")));
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(1)->AwaitMutualSyncCycleCompletion(GetClient(0)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
const BookmarkNode* new_folder = AddFolder(0, 2, "New Folder");
Move(0, GetUniqueNodeByURL(0, google_url), new_folder, 0);
SetTitle(0, GetBookmarkBarNode(0)->GetChild(0), "Yahoo!!");
ASSERT_NE(nullptr, AddURL(0, GetBookmarkBarNode(0), 1, "CNN",
GURL("http://www.cnn.com")));
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
ASSERT_NE(nullptr, AddURL(1, "Facebook", GURL("http://www.facebook.com")));
-
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(1)->AwaitMutualSyncCycleCompletion(GetClient(0)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
SortChildren(1, GetBookmarkBarNode(1));
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(1)->AwaitMutualSyncCycleCompletion(GetClient(0)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
DisableVerifier();
SetTitle(0, GetUniqueNodeByURL(0, google_url), "Google++");
SetTitle(1, GetUniqueNodeByURL(1, google_url), "Google--");
- ASSERT_TRUE(BookmarksMatchChecker().Wait());
+ ASSERT_TRUE(AwaitQuiescence());
+ ASSERT_TRUE(AllModelsMatch());
}
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, SimultaneousURLChanges) {
@@ -146,10 +150,12 @@
DisableVerifier();
ASSERT_NE(nullptr, SetURL(0, GetUniqueNodeByURL(0, initial_url), second_url));
ASSERT_NE(nullptr, SetURL(1, GetUniqueNodeByURL(1, initial_url), third_url));
- ASSERT_TRUE(BookmarksMatchChecker().Wait());
+ ASSERT_TRUE(AwaitQuiescence());
+ ASSERT_TRUE(AllModelsMatch());
SetTitle(0, GetBookmarkBarNode(0)->GetChild(0), "Google1");
- ASSERT_TRUE(BookmarksMatchChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatch());
}
// Test Scribe ID - 370558.
@@ -158,7 +164,8 @@
ASSERT_TRUE(AllModelsMatchVerifier());
ASSERT_NE(nullptr, AddFolder(0, kGenericFolderName));
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
}
// Test Scribe ID - 370559.
@@ -168,22 +175,26 @@
ASSERT_TRUE(AllModelsMatchVerifier());
ASSERT_NE(nullptr, AddURL(0, kGenericURLTitle, GURL(kGenericURL)));
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
}
// Test Scribe ID - 370489.
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, SC_AddFirstBMWithFavicon) {
ASSERT_TRUE(SetupSync()) << "SetupSync() failed.";
ASSERT_TRUE(AllModelsMatchVerifier());
+
const GURL page_url(kGenericURL);
const GURL icon_url("http://www.google.com/favicon.ico");
const BookmarkNode* bookmark = AddURL(0, kGenericURLTitle, page_url);
+
ASSERT_NE(nullptr, bookmark);
ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
SetFavicon(0, bookmark, icon_url, CreateFavicon(SK_ColorWHITE),
bookmarks_helper::FROM_UI);
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
}
// Test that the history service logic for not losing the hidpi versions of
@@ -212,16 +223,19 @@
ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
SetFavicon(0, bookmark0, icon_url1, CreateFavicon(SK_ColorWHITE),
bookmarks_helper::FROM_UI);
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
const BookmarkNode* bookmark1 = GetUniqueNodeByURL(1, page_url);
SetFavicon(1, bookmark1, icon_url1, CreateFavicon(SK_ColorBLUE),
bookmarks_helper::FROM_UI);
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(1)->AwaitMutualSyncCycleCompletion(GetClient(0)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
SetFavicon(0, bookmark0, icon_url2, CreateFavicon(SK_ColorGREEN),
bookmarks_helper::FROM_UI);
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
}
// Test that if sync does not modify a favicon bitmap's data that it does not
@@ -255,7 +269,8 @@
ExpireFavicon(0, bookmark0);
CheckFaviconExpired(0, icon_url);
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
// Change the bookmark's title for profile 1. Changing the title will cause
// the bookmark's favicon data to be synced from profile 1 to profile 0 even
@@ -264,7 +279,8 @@
ASSERT_NE(kNewTitle, kGenericURLTitle);
const BookmarkNode* bookmark1 = GetUniqueNodeByURL(1, page_url);
SetTitle(1, bookmark1, kNewTitle);
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(1)->AwaitMutualSyncCycleCompletion(GetClient(0)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
// The favicon for profile 0 should still be expired.
CheckFaviconExpired(0, icon_url);
@@ -292,13 +308,15 @@
SetFavicon(0, bookmark01, icon_url, CreateFavicon(SK_ColorWHITE),
bookmarks_helper::FROM_UI);
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
// Set |page_url2| with the new (blue) favicon at |icon_url|. The sync favicon
// for both |page_url1| and |page_url2| should be updated to the blue one.
SetFavicon(0, bookmark02, icon_url, CreateFavicon(SK_ColorBLUE),
bookmarks_helper::FROM_UI);
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
// Set the title for |page_url1|. This should not revert either of the
// bookmark favicons back to white.
@@ -306,7 +324,8 @@
ASSERT_STRNE(kGenericURLTitle, kNewTitle);
const BookmarkNode* bookmark11 = GetUniqueNodeByURL(1, page_url1);
SetTitle(1, bookmark11, std::string(kNewTitle));
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(1)->AwaitMutualSyncCycleCompletion(GetClient(0)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
}
// Test Scribe ID - 370560.
@@ -317,7 +336,8 @@
ASSERT_NE(nullptr,
AddURL(0, "FTP UR", GURL("ftp://user:password@host:1234/path")));
ASSERT_NE(nullptr, AddURL(0, "File UR", GURL("file://host/path")));
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
}
// Test Scribe ID - 370561.
@@ -329,7 +349,8 @@
const BookmarkNode* folder = AddFolder(0, kGenericFolderName);
ASSERT_NE(nullptr, folder);
ASSERT_NE(nullptr, AddURL(0, folder, 0, kGenericURLTitle, GURL(kGenericURL)));
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
}
// Test Scribe ID - 370562.
@@ -344,7 +365,8 @@
ASSERT_NE(nullptr, AddURL(0, i, title, url));
ASSERT_NE(nullptr, AddURL(0, GetOtherNode(0), i, title, url));
}
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
}
// Test Scribe ID - 370563.
@@ -376,7 +398,8 @@
GURL url = GURL(IndexedURL(i));
ASSERT_NE(nullptr, AddURL(0, GetOtherNode(0), i, title, url));
}
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
}
// Test Scribe ID - 370641.
@@ -389,7 +412,8 @@
GURL url1 = GURL(IndexedURL(1));
ASSERT_NE(nullptr, AddURL(0, kGenericURLTitle, url0));
ASSERT_NE(nullptr, AddURL(0, kGenericURLTitle, url1));
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
}
// Test Scribe ID - 370639 - Add bookmarks with different name and same URL.
@@ -402,7 +426,8 @@
std::string title1 = IndexedURLTitle(1);
ASSERT_NE(nullptr, AddURL(0, title0, GURL(kGenericURL)));
ASSERT_NE(nullptr, AddURL(0, title1, GURL(kGenericURL)));
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
}
// Test Scribe ID - 371817.
@@ -413,11 +438,13 @@
std::string title = IndexedURLTitle(1);
const BookmarkNode* bookmark = AddURL(0, title, GURL(kGenericURL));
ASSERT_NE(nullptr, bookmark);
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
std::string new_title = IndexedURLTitle(2);
SetTitle(0, bookmark, new_title);
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
}
// Test Scribe ID - 371822.
@@ -428,11 +455,13 @@
GURL url = GURL(IndexedURL(1));
const BookmarkNode* bookmark = AddURL(0, kGenericURLTitle, url);
ASSERT_NE(nullptr, bookmark);
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
GURL new_url = GURL(IndexedURL(2));
ASSERT_NE(nullptr, SetURL(0, bookmark, new_url));
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
}
@@ -445,14 +474,17 @@
std::string title = IndexedURLTitle(1);
const BookmarkNode* bookmark = AddURL(0, title, GURL(kGenericURL));
ASSERT_NE(nullptr, bookmark);
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
std::string new_title = IndexedURLTitle(2);
SetTitle(0, bookmark, new_title);
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
SetTitle(0, bookmark, title);
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
}
// Test Scribe ID - 371823 - Renaming the same bookmark URL twice.
@@ -464,14 +496,17 @@
GURL url = GURL(IndexedURL(1));
const BookmarkNode* bookmark = AddURL(0, kGenericURLTitle, url);
ASSERT_NE(nullptr, bookmark);
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
GURL new_url = GURL(IndexedURL(2));
ASSERT_NE(nullptr, SetURL(0, bookmark, new_url));
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
ASSERT_NE(nullptr, SetURL(0, bookmark, url));
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
}
// Test Scribe ID - 371824.
@@ -482,11 +517,13 @@
std::string title = IndexedFolderName(1);
const BookmarkNode* folder = AddFolder(0, title);
ASSERT_NE(nullptr, AddURL(0, folder, 0, kGenericURLTitle, GURL(kGenericURL)));
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
std::string new_title = IndexedFolderName(2);
SetTitle(0, folder, new_title);
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
}
// Test Scribe ID - 371825.
@@ -496,11 +533,13 @@
std::string title = IndexedFolderName(1);
const BookmarkNode* folder = AddFolder(0, title);
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
std::string new_title = IndexedFolderName(2);
SetTitle(0, folder, new_title);
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
}
// Test Scribe ID - 371826.
@@ -522,11 +561,13 @@
ASSERT_NE(nullptr, AddFolder(0, folder, i, title));
}
}
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
std::string new_title = IndexedFolderName(2);
SetTitle(0, folder, new_title);
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
}
// Test Scribe ID - 371827.
@@ -554,11 +595,13 @@
ASSERT_NE(nullptr, AddFolder(0, subfolder, i, title));
}
}
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
std::string new_title = IndexedSubfolderName(2);
SetTitle(0, subfolder, new_title);
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
}
// Test Scribe ID - 371828.
@@ -570,14 +613,16 @@
std::string title = IndexedURLTitle(1);
const BookmarkNode* bookmark = AddURL(0, title, url);
ASSERT_NE(nullptr, bookmark);
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
GURL new_url = GURL(IndexedURL(2));
std::string new_title = IndexedURLTitle(2);
bookmark = SetURL(0, bookmark, new_url);
ASSERT_NE(nullptr, bookmark);
SetTitle(0, bookmark, new_title);
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
}
// Test Scribe ID - 371832.
@@ -587,10 +632,12 @@
ASSERT_TRUE(AllModelsMatchVerifier());
ASSERT_NE(nullptr, AddURL(0, kGenericURLTitle, GURL(kGenericURL)));
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
Remove(0, GetBookmarkBarNode(0), 0);
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
}
// Test Scribe ID - 371833.
@@ -604,10 +651,12 @@
GURL url = GURL(IndexedURL(i));
ASSERT_NE(nullptr, AddURL(0, i, title, url));
}
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
Remove(0, GetBookmarkBarNode(0), 0);
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
}
// Test Scribe ID - 371835.
@@ -623,10 +672,12 @@
GURL url = GURL(IndexedURL(i));
ASSERT_NE(nullptr, AddURL(0, folder, i, title, url));
}
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
Remove(0, folder, 0);
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
}
// Test Scribe ID - 371836.
@@ -642,10 +693,12 @@
GURL url = GURL(IndexedURL(i));
ASSERT_NE(nullptr, AddURL(0, folder, i, title, url));
}
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
Remove(0, folder, folder->child_count() - 1);
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
}
// Test Scribe ID - 371856.
@@ -661,10 +714,12 @@
GURL url = GURL(IndexedURL(i));
ASSERT_NE(nullptr, AddURL(0, folder, i, title, url));
}
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
Remove(0, folder, 4);
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
}
// Test Scribe ID - 371857.
@@ -680,13 +735,15 @@
GURL url = GURL(IndexedURL(i));
ASSERT_NE(nullptr, AddURL(0, folder, i, title, url));
}
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
int child_count = folder->child_count();
for (int i = 0; i < child_count; ++i) {
Remove(0, folder, 0);
}
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
}
// Test Scribe ID - 371858.
@@ -696,10 +753,12 @@
ASSERT_TRUE(AllModelsMatchVerifier());
ASSERT_NE(nullptr, AddFolder(0, kGenericFolderName));
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
Remove(0, GetBookmarkBarNode(0), 0);
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
}
// Test Scribe ID - 371869.
@@ -719,10 +778,12 @@
ASSERT_NE(nullptr, AddFolder(0, i, title));
}
}
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
Remove(0, GetBookmarkBarNode(0), 0);
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
}
// Test Scribe ID - 371879.
@@ -749,10 +810,12 @@
GURL url = GURL(IndexedURL(i));
ASSERT_NE(nullptr, AddURL(0, folder, i, title, url));
}
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
Remove(0, GetBookmarkBarNode(0), 1);
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
}
// Test Scribe ID - 371880.
@@ -798,10 +861,12 @@
}
}
}
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
Remove(0, GetBookmarkBarNode(0), 1);
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
}
// Test Scribe ID - 371882.
@@ -830,10 +895,12 @@
ASSERT_NE(nullptr, AddFolder(0, subfolder, i, title));
}
}
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
Remove(0, folder, 0);
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
}
// Test Scribe ID - 371931.
@@ -850,10 +917,12 @@
const BookmarkNode* bookmark1 = AddURL(0, 1, title1, url1);
ASSERT_NE(nullptr, bookmark0);
ASSERT_NE(nullptr, bookmark1);
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
Move(0, bookmark0, GetBookmarkBarNode(0), 2);
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
}
// Test Scribe ID - 371933.
@@ -866,10 +935,12 @@
GURL url = GURL(IndexedURL(i));
ASSERT_NE(nullptr, AddURL(0, i, title, url));
}
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
ReverseChildOrder(0, GetBookmarkBarNode(0));
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
}
// Test Scribe ID - 371954.
@@ -893,7 +964,8 @@
GURL url = GURL(IndexedURL(i));
ASSERT_NE(nullptr, AddURL(0, i, title, url));
}
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
int num_bookmarks_to_move =
GetBookmarkBarNode(0)->child_count() - 2;
@@ -926,7 +998,8 @@
GURL url = GURL(IndexedURL(i));
ASSERT_NE(nullptr, AddURL(0, folder, i, title, url));
}
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
int num_bookmarks_to_move = folder->child_count() - 2;
for (int i = 0; i < num_bookmarks_to_move; ++i) {
@@ -957,13 +1030,15 @@
GURL url = GURL(IndexedURL(i + 3));
ASSERT_NE(nullptr, AddURL(0, subfolder, i, title, url));
}
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
for (int i = 0; i < 3; ++i) {
GURL url = GURL(IndexedURL(i));
Move(0, GetUniqueNodeByURL(0, url), subfolder, i + 10);
}
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
}
// Test Scribe ID - 371964.
@@ -987,13 +1062,15 @@
GURL url = GURL(IndexedURL(i + 3));
ASSERT_NE(nullptr, AddURL(0, subfolder, i, title, url));
}
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
for (int i = 0; i < 3; ++i) {
GURL url = GURL(IndexedURL(i + 3));
Move(0, GetUniqueNodeByURL(0, url), folder, i + 4);
}
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
}
// Test Scribe ID - 371967.
@@ -1022,7 +1099,8 @@
GURL url = GURL(IndexedURL(i + 10));
ASSERT_NE(nullptr, AddURL(0, folder_L10, i, title, url));
}
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
GURL url10 = GURL(IndexedURL(10));
Move(0, GetUniqueNodeByURL(
@@ -1031,7 +1109,8 @@
Move(0, GetUniqueNodeByURL(0, url11), folder_L0, 0);
GURL url12 = GURL(IndexedURL(12));
Move(0, GetUniqueNodeByURL(0, url12), folder_L0, 1);
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
}
// Test Scribe ID - 371968.
@@ -1061,7 +1140,8 @@
GURL url = GURL(IndexedURL(i + 10));
ASSERT_NE(nullptr, AddURL(0, folder_L0, 0, title, url));
}
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
GURL url10 = GURL(IndexedURL(10));
Move(0, GetUniqueNodeByURL(0, url10), folder_L10, folder_L10->child_count());
@@ -1069,7 +1149,8 @@
Move(0, GetUniqueNodeByURL(0, url11), folder_L10, 0);
GURL url12 = GURL(IndexedURL(12));
Move(0, GetUniqueNodeByURL(0, url12), folder_L10, 1);
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
}
// Test Scribe ID - 371980.
@@ -1096,10 +1177,12 @@
folder = AddFolder(
0, GetBookmarkBarNode(0)->child_count(), kGenericFolderName);
ASSERT_NE(nullptr, folder);
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
Move(0, folder, folder_L5, folder_L5->child_count());
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
}
// Test Scribe ID - 371997.
@@ -1130,10 +1213,12 @@
GURL url = GURL(IndexedURL(i));
ASSERT_NE(nullptr, AddURL(0, folder, i, title, url));
}
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
Move(0, folder, folder_L5, folder_L5->child_count());
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
}
// Test Scribe ID - 372006.
@@ -1164,10 +1249,12 @@
GURL url = GURL(IndexedURL(i));
ASSERT_NE(nullptr, AddURL(0, folder, i, title, url));
}
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
Move(0, folder, GetBookmarkBarNode(0), GetBookmarkBarNode(0)->child_count());
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
}
// Test Scribe ID - 372026.
@@ -1186,10 +1273,12 @@
ASSERT_NE(nullptr, AddURL(0, folder, j, title, url));
}
}
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
ReverseChildOrder(0, GetBookmarkBarNode(0));
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
}
// Test Scribe ID - 372028.
@@ -1208,10 +1297,12 @@
ASSERT_NE(nullptr, AddURL(0, folder, j, title, url));
}
}
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
ReverseChildOrder(0, GetBookmarkBarNode(0));
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
}
// Test Scribe ID - 373379.
@@ -1229,7 +1320,8 @@
GURL url1 = GURL(IndexedURL(2*i+1));
ASSERT_NE(nullptr, AddURL(1, title1, url1));
}
- ASSERT_TRUE(BookmarksMatchChecker().Wait());
+ ASSERT_TRUE(AwaitQuiescence());
+ ASSERT_TRUE(AllModelsMatch());
ASSERT_FALSE(ContainsDuplicateBookmarks(0));
}
@@ -1248,7 +1340,8 @@
ASSERT_NE(nullptr, AddURL(0, title, url));
ASSERT_NE(nullptr, AddURL(1, title, url));
}
- ASSERT_TRUE(BookmarksMatchChecker().Wait());
+ ASSERT_TRUE(AwaitQuiescence());
+ ASSERT_TRUE(AllModelsMatch());
}
// Test Scribe ID - 373506.
@@ -1278,7 +1371,8 @@
ASSERT_NE(nullptr, AddURL(1, folder1, 2, "bOOKMARK 2", GURL(kGenericURL)));
ASSERT_TRUE(SetupSync()) << "SetupSync() failed.";
- ASSERT_TRUE(BookmarksMatchChecker().Wait());
+ ASSERT_TRUE(AwaitQuiescence());
+ ASSERT_TRUE(AllModelsMatch());
ASSERT_FALSE(ContainsDuplicateBookmarks(0));
}
@@ -1305,7 +1399,8 @@
}
ASSERT_TRUE(SetupSync()) << "SetupSync() failed.";
- ASSERT_TRUE(BookmarksMatchChecker().Wait());
+ ASSERT_TRUE(AwaitQuiescence());
+ ASSERT_TRUE(AllModelsMatch());
ASSERT_FALSE(ContainsDuplicateBookmarks(0));
}
@@ -1329,7 +1424,8 @@
}
ASSERT_TRUE(SetupSync()) << "SetupSync() failed.";
- ASSERT_TRUE(BookmarksMatchChecker().Wait());
+ ASSERT_TRUE(AwaitQuiescence());
+ ASSERT_TRUE(AllModelsMatch());
ASSERT_FALSE(ContainsDuplicateBookmarks(0));
}
@@ -1347,7 +1443,8 @@
}
ASSERT_TRUE(SetupSync()) << "SetupSync() failed.";
- ASSERT_TRUE(BookmarksMatchChecker().Wait());
+ ASSERT_TRUE(AwaitQuiescence());
+ ASSERT_TRUE(AllModelsMatch());
ASSERT_FALSE(ContainsDuplicateBookmarks(0));
}
@@ -1370,7 +1467,8 @@
ASSERT_NE(nullptr, AddURL(1, folder1, i, title1, url1));
}
ASSERT_TRUE(SetupSync()) << "SetupSync() failed.";
- ASSERT_TRUE(BookmarksMatchChecker().Wait());
+ ASSERT_TRUE(AwaitQuiescence());
+ ASSERT_TRUE(AllModelsMatch());
ASSERT_FALSE(ContainsDuplicateBookmarks(0));
}
@@ -1411,7 +1509,8 @@
ASSERT_NE(nullptr, AddURL(1, title, url));
}
ASSERT_TRUE(SetupSync()) << "SetupSync() failed.";
- ASSERT_TRUE(BookmarksMatchChecker().Wait());
+ ASSERT_TRUE(AwaitQuiescence());
+ ASSERT_TRUE(AllModelsMatch());
ASSERT_FALSE(ContainsDuplicateBookmarks(0));
}
@@ -1436,7 +1535,8 @@
}
}
ASSERT_TRUE(SetupSync()) << "SetupSync() failed.";
- ASSERT_TRUE(BookmarksMatchChecker().Wait());
+ ASSERT_TRUE(AwaitQuiescence());
+ ASSERT_TRUE(AllModelsMatch());
ASSERT_FALSE(ContainsDuplicateBookmarks(0));
}
@@ -1459,7 +1559,8 @@
}
ASSERT_TRUE(SetupSync()) << "SetupSync() failed.";
- ASSERT_TRUE(BookmarksMatchChecker().Wait());
+ ASSERT_TRUE(AwaitQuiescence());
+ ASSERT_TRUE(AllModelsMatch());
ASSERT_FALSE(ContainsDuplicateBookmarks(0));
ASSERT_FALSE(ContainsDuplicateBookmarks(1));
}
@@ -1490,7 +1591,8 @@
AddURL(1, folder1, 4, IndexedURLTitle(1), GURL(IndexedURL(1))));
ASSERT_TRUE(SetupSync()) << "SetupSync() failed.";
- ASSERT_TRUE(BookmarksMatchChecker().Wait());
+ ASSERT_TRUE(AwaitQuiescence());
+ ASSERT_TRUE(AllModelsMatch());
ASSERT_FALSE(ContainsDuplicateBookmarks(0));
}
@@ -1520,7 +1622,8 @@
AddURL(1, folder1, 3, IndexedURLTitle(8), GURL(IndexedURL(8))));
ASSERT_TRUE(SetupSync()) << "SetupSync() failed.";
- ASSERT_TRUE(BookmarksMatchChecker().Wait());
+ ASSERT_TRUE(AwaitQuiescence());
+ ASSERT_TRUE(AllModelsMatch());
ASSERT_FALSE(ContainsDuplicateBookmarks(0));
}
@@ -1544,7 +1647,8 @@
}
ASSERT_TRUE(SetupSync()) << "SetupSync() failed.";
- ASSERT_TRUE(BookmarksMatchChecker().Wait());
+ ASSERT_TRUE(AwaitQuiescence());
+ ASSERT_TRUE(AllModelsMatch());
ASSERT_FALSE(ContainsDuplicateBookmarks(0));
}
@@ -1572,7 +1676,8 @@
}
ASSERT_TRUE(SetupSync()) << "SetupSync() failed.";
- ASSERT_TRUE(BookmarksMatchChecker().Wait());
+ ASSERT_TRUE(AwaitQuiescence());
+ ASSERT_TRUE(AllModelsMatch());
for (int i = 1; i < 5 ; ++i) {
ASSERT_EQ(i, CountBookmarksWithTitlesMatching(1, IndexedURLTitle(i)));
@@ -1590,7 +1695,8 @@
ASSERT_FALSE(AllModelsMatch());
ASSERT_TRUE(GetClient(1)->EnableSyncForDatatype(syncer::BOOKMARKS));
- ASSERT_TRUE(BookmarksMatchChecker().Wait());
+ ASSERT_TRUE(AwaitQuiescence());
+ ASSERT_TRUE(AllModelsMatch());
}
// TCM ID - 7343544.
@@ -1607,7 +1713,8 @@
ASSERT_FALSE(AllModelsMatch());
ASSERT_TRUE(GetClient(1)->EnableSyncForAllDatatypes());
- ASSERT_TRUE(BookmarksMatchChecker().Wait());
+ ASSERT_TRUE(AwaitQuiescence());
+ ASSERT_TRUE(AllModelsMatch());
}
// TCM ID - 3662298 - Test adding duplicate folder - Both with different BMs
@@ -1630,7 +1737,8 @@
}
ASSERT_TRUE(SetupSync()) << "SetupSync() failed.";
- ASSERT_TRUE(BookmarksMatchChecker().Wait());
+ ASSERT_TRUE(AwaitQuiescence());
+ ASSERT_TRUE(AllModelsMatch());
ASSERT_FALSE(ContainsDuplicateBookmarks(0));
}
@@ -1680,7 +1788,8 @@
GURL url = GURL(IndexedURL(i));
ASSERT_NE(nullptr, AddURL(0, folder0, i, title, url));
}
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
ASSERT_FALSE(ContainsDuplicateBookmarks(0));
DisableVerifier();
@@ -1688,7 +1797,8 @@
SetTitle(0, GetUniqueNodeByURL(0, url), "Title++");
SetTitle(1, GetUniqueNodeByURL(1, url), "Title--");
- ASSERT_TRUE(BookmarksMatchChecker().Wait());
+ ASSERT_TRUE(AwaitQuiescence());
+ ASSERT_TRUE(AllModelsMatch());
ASSERT_FALSE(ContainsDuplicateBookmarks(0));
}
@@ -1705,7 +1815,8 @@
GURL url = GURL(IndexedURL(i));
ASSERT_NE(nullptr, AddURL(0, folder0, i, title, url));
}
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
ASSERT_FALSE(ContainsDuplicateBookmarks(0));
DisableVerifier();
@@ -1715,7 +1826,8 @@
ASSERT_TRUE(SetURL(
1, GetUniqueNodeByURL(1, url), GURL("http://www.google.com/11")));
- ASSERT_TRUE(BookmarksMatchChecker().Wait());
+ ASSERT_TRUE(AwaitQuiescence());
+ ASSERT_TRUE(AllModelsMatch());
ASSERT_FALSE(ContainsDuplicateBookmarks(0));
}
@@ -1764,25 +1876,29 @@
}
ASSERT_TRUE(SetupSync()) << "SetupSync() failed.";
- ASSERT_TRUE(BookmarksMatchChecker().Wait());
+ ASSERT_TRUE(AwaitQuiescence());
+ ASSERT_TRUE(AllModelsMatch());
ASSERT_FALSE(ContainsDuplicateBookmarks(0));
// Simultaneously rename folder A on both clients.
SetTitle(0, folderA[0], "Folder A++");
SetTitle(1, folderA[1], "Folder A--");
- ASSERT_TRUE(BookmarksMatchChecker().Wait());
+ ASSERT_TRUE(AwaitQuiescence());
+ ASSERT_TRUE(AllModelsMatch());
ASSERT_FALSE(ContainsDuplicateBookmarks(0));
// Simultaneously rename folder B on both clients.
SetTitle(0, folderB[0], "Folder B++");
SetTitle(1, folderB[1], "Folder B--");
- ASSERT_TRUE(BookmarksMatchChecker().Wait());
+ ASSERT_TRUE(AwaitQuiescence());
+ ASSERT_TRUE(AllModelsMatch());
ASSERT_FALSE(ContainsDuplicateBookmarks(0));
// Simultaneously rename folder C on both clients.
SetTitle(0, folderC[0], "Folder C++");
SetTitle(1, folderC[1], "Folder C--");
- ASSERT_TRUE(BookmarksMatchChecker().Wait());
+ ASSERT_TRUE(AwaitQuiescence());
+ ASSERT_TRUE(AllModelsMatch());
ASSERT_FALSE(ContainsDuplicateBookmarks(0));
}
@@ -1855,7 +1971,8 @@
ASSERT_NE(nullptr, AddURL(0, IndexedURLTitle(1), GURL(IndexedURL(1))));
ASSERT_NE(nullptr, AddFolder(0, IndexedFolderName(0)));
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
}
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest,
@@ -1867,7 +1984,8 @@
ASSERT_NE(nullptr, AddURL(0, 0, IndexedURLTitle(0), GURL(IndexedURL(0))));
ASSERT_NE(nullptr, AddURL(0, 1, IndexedURLTitle(1), GURL(IndexedURL(1))));
ASSERT_NE(nullptr, AddURL(0, 2, IndexedURLTitle(2), GURL(IndexedURL(2))));
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
// Set a passphrase and enable encryption on Client 0. Client 1 will not
// understand the bookmark updates.
@@ -1890,12 +2008,14 @@
ASSERT_TRUE(PassphraseRequiredChecker(GetSyncService(1)).Wait());
ASSERT_TRUE(GetSyncService(1)->SetDecryptionPassphrase(kValidPassphrase));
ASSERT_TRUE(PassphraseAcceptedChecker(GetSyncService(1)).Wait());
- ASSERT_TRUE(BookmarksMatchChecker().Wait());
+ ASSERT_TRUE(AwaitQuiescence());
+ EXPECT_TRUE(AllModelsMatch());
ASSERT_EQ(0, GetClient(1)->GetLastCycleSnapshot().num_encryption_conflicts());
// Ensure everything is syncing normally by appending a final bookmark.
ASSERT_NE(nullptr, AddURL(1, 5, IndexedURLTitle(5), GURL(IndexedURL(5))));
- ASSERT_TRUE(BookmarksMatchChecker().Wait());
+ ASSERT_TRUE(GetClient(1)->AwaitMutualSyncCycleCompletion(GetClient(0)));
+ EXPECT_TRUE(AllModelsMatch());
ASSERT_EQ(0, GetClient(1)->GetLastCycleSnapshot().num_encryption_conflicts());
}
@@ -1935,7 +2055,8 @@
Move(1, node, node->parent(), rand_pos);
}
- ASSERT_TRUE(BookmarksMatchChecker().Wait());
+ ASSERT_TRUE(AwaitQuiescence());
+ ASSERT_TRUE(AllModelsMatch());
// Now make changes to client 1 first.
for (size_t i = 0; i < num_bookmarks; ++i) {
@@ -1955,7 +2076,8 @@
Move(0, node, node->parent(), rand_pos);
}
- ASSERT_TRUE(BookmarksMatchChecker().Wait());
+ ASSERT_TRUE(AwaitQuiescence());
+ ASSERT_TRUE(AllModelsMatch());
}
// Trigger the server side creation of Synced Bookmarks. Ensure both clients
@@ -1971,7 +2093,8 @@
// Add a bookmark on Client 0 and ensure it syncs over. This will also trigger
// both clients downloading the new Synced Bookmarks folder.
ASSERT_NE(nullptr, AddURL(0, "Google", GURL("http://www.google.com")));
- ASSERT_TRUE(BookmarksMatchChecker().Wait());
+ ASSERT_TRUE(AwaitQuiescence());
+ ASSERT_TRUE(AllModelsMatch());
// Now add a bookmark within the Synced Bookmarks folder and ensure it syncs
// over.
@@ -1979,7 +2102,8 @@
ASSERT_TRUE(synced_bookmarks);
ASSERT_NE(nullptr, AddURL(0, synced_bookmarks, 0, "Google2",
GURL("http://www.google2.com")));
- ASSERT_TRUE(BookmarksMatchChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatch());
}
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest,
@@ -2023,7 +2147,8 @@
ASSERT_TRUE(
AddURL(0, GetBookmarkBarNode(0), 2, "Gmai", GURL("http://gmail.com")));
- ASSERT_TRUE(BookmarksMatchChecker().Wait());
+ ASSERT_TRUE(AwaitQuiescence());
+ ASSERT_TRUE(AllModelsMatch());
// Remove all
RemoveAll(0);
@@ -2067,7 +2192,8 @@
// Verify that adding a bookmark is observed by the second Profile.
GURL google_url("http://www.google.com");
ASSERT_NE(nullptr, AddURL(0, "Google", google_url));
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
ASSERT_EQ(1, bar_node0->child_count());
ASSERT_EQ(1, bar_node1->child_count());
@@ -2087,7 +2213,8 @@
// Now add another user bookmark and wait for it to sync.
ASSERT_NE(nullptr, AddURL(0, "Google 2", google_url));
- ASSERT_TRUE(BookmarksMatchVerifierChecker().Wait());
+ ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
+ ASSERT_TRUE(AllModelsMatchVerifier());
EXPECT_FALSE(GetSyncService(0)->HasUnrecoverableError());
EXPECT_FALSE(GetSyncService(1)->HasUnrecoverableError());
« no previous file with comments | « chrome/browser/sync/test/integration/sync_test.cc ('k') | components/sync/driver/glue/sync_backend_host_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698