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

Unified Diff: components/sync/driver/glue/sync_backend_host_impl_unittest.cc

Issue 2644373003: Revert of [Sync] Make directory types registration explicit in ModelTypeRegistry (Closed)
Patch Set: Created 3 years, 11 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: components/sync/driver/glue/sync_backend_host_impl_unittest.cc
diff --git a/components/sync/driver/glue/sync_backend_host_impl_unittest.cc b/components/sync/driver/glue/sync_backend_host_impl_unittest.cc
index a96b4b2baa09ff3fcb713c048ac70e95063e0090..21d8d42d380367937db990b2a8df5487792ae1dd 100644
--- a/components/sync/driver/glue/sync_backend_host_impl_unittest.cc
+++ b/components/sync/driver/glue/sync_backend_host_impl_unittest.cc
@@ -323,6 +323,7 @@
EXPECT_TRUE(fake_manager_->GetAndResetDownloadedTypes().HasAll(
Difference(enabled_types_, ControlTypes())));
EXPECT_EQ(enabled_types_, fake_manager_->InitialSyncEndedTypes());
+ EXPECT_EQ(enabled_types_, fake_manager_->GetAndResetEnabledTypes());
EXPECT_TRUE(
fake_manager_->GetTypesWithEmptyProgressMarkerToken(enabled_types_)
.Empty());
@@ -352,6 +353,7 @@
Intersection(fake_manager_->GetAndResetPurgedTypes(), enabled_types_)
.Empty());
EXPECT_EQ(enabled_types_, fake_manager_->InitialSyncEndedTypes());
+ EXPECT_EQ(enabled_types_, fake_manager_->GetAndResetEnabledTypes());
EXPECT_TRUE(
fake_manager_->GetTypesWithEmptyProgressMarkerToken(enabled_types_)
.Empty());
@@ -387,6 +389,7 @@
.Empty());
EXPECT_EQ(partial_types, fake_manager_->GetAndResetDownloadedTypes());
EXPECT_EQ(enabled_types_, fake_manager_->InitialSyncEndedTypes());
+ EXPECT_EQ(enabled_types_, fake_manager_->GetAndResetEnabledTypes());
EXPECT_TRUE(
fake_manager_->GetTypesWithEmptyProgressMarkerToken(enabled_types_)
.Empty());
@@ -421,6 +424,7 @@
Intersection(fake_manager_->GetAndResetPurgedTypes(), enabled_types_)
.Empty());
EXPECT_EQ(enabled_types_, fake_manager_->InitialSyncEndedTypes());
+ EXPECT_EQ(enabled_types_, fake_manager_->GetAndResetEnabledTypes());
EXPECT_TRUE(
fake_manager_->GetTypesWithEmptyProgressMarkerToken(enabled_types_)
.Empty());
@@ -455,6 +459,7 @@
EXPECT_EQ(disabled_types,
Intersection(fake_manager_->GetAndResetPurgedTypes(), old_types));
EXPECT_EQ(enabled_types_, fake_manager_->InitialSyncEndedTypes());
+ EXPECT_EQ(enabled_types_, fake_manager_->GetAndResetEnabledTypes());
EXPECT_TRUE(
fake_manager_->GetTypesWithEmptyProgressMarkerToken(enabled_types_)
.Empty());
@@ -490,6 +495,7 @@
Intersection(fake_manager_->GetAndResetPurgedTypes(), enabled_types_)
.Empty());
EXPECT_EQ(enabled_types_, fake_manager_->InitialSyncEndedTypes());
+ EXPECT_EQ(enabled_types_, fake_manager_->GetAndResetEnabledTypes());
EXPECT_TRUE(
fake_manager_->GetTypesWithEmptyProgressMarkerToken(enabled_types_)
.Empty());
@@ -528,6 +534,7 @@
EXPECT_EQ(disabled_types,
Intersection(fake_manager_->GetAndResetPurgedTypes(), old_types));
EXPECT_EQ(enabled_types_, fake_manager_->InitialSyncEndedTypes());
+ EXPECT_EQ(enabled_types_, fake_manager_->GetAndResetEnabledTypes());
EXPECT_EQ(disabled_types,
fake_manager_->GetTypesWithEmptyProgressMarkerToken(old_types));
}
@@ -563,6 +570,7 @@
Intersection(fake_manager_->GetAndResetPurgedTypes(), enabled_types_)
.Empty());
EXPECT_EQ(enabled_types_, fake_manager_->InitialSyncEndedTypes());
+ EXPECT_EQ(enabled_types_, fake_manager_->GetAndResetEnabledTypes());
EXPECT_TRUE(
fake_manager_->GetTypesWithEmptyProgressMarkerToken(enabled_types_)
.Empty());
@@ -604,6 +612,7 @@
Intersection(fake_manager_->GetAndResetPurgedTypes(), enabled_types_)
.Empty());
EXPECT_EQ(enabled_types_, fake_manager_->InitialSyncEndedTypes());
+ EXPECT_EQ(enabled_types_, fake_manager_->GetAndResetEnabledTypes());
EXPECT_TRUE(
fake_manager_->GetTypesWithEmptyProgressMarkerToken(enabled_types_)
.Empty());
« no previous file with comments | « components/sync/driver/glue/sync_backend_host_impl.cc ('k') | components/sync/driver/model_association_manager.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698