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

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

Issue 118463002: Sync removal of Default apps. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Don't BuildModel() if no extension service. Created 7 years 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/browser/sync/test/integration/sync_app_list_helper.cc
diff --git a/chrome/browser/sync/test/integration/sync_app_list_helper.cc b/chrome/browser/sync/test/integration/sync_app_list_helper.cc
index 1ae254309a93a33ce2f9b23877c088b73e50be3b..2a6855a028d29c00f9849ba77bb9dbae0dfa5e1e 100644
--- a/chrome/browser/sync/test/integration/sync_app_list_helper.cc
+++ b/chrome/browser/sync/test/integration/sync_app_list_helper.cc
@@ -54,13 +54,8 @@ bool SyncAppListHelper::AppListMatchesVerifier(Profile* profile) {
AppListSyncableServiceFactory::GetForProfile(profile);
AppListSyncableService* verifier =
AppListSyncableServiceFactory::GetForProfile(test_->verifier());
- if (service->GetNumSyncItemsForTest() !=
- verifier->GetNumSyncItemsForTest()) {
- LOG(ERROR) << "Sync item count: "
- << service->GetNumSyncItemsForTest()
- << " != " << verifier->GetNumSyncItemsForTest();
- return false;
- }
+ // Note: sync item entries may not exist in verifier, but item lists should
+ // match.
if (service->model()->item_list()->item_count() !=
verifier->model()->item_list()->item_count()) {
LOG(ERROR) << "Model item count: "

Powered by Google App Engine
This is Rietveld 408576698