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

Unified Diff: chrome/browser/sync/glue/sync_backend_registrar_unittest.cc

Issue 17552014: [Sync] Have SBH tell SyncManager which types to purge (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix dchecks Created 7 years, 6 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
« no previous file with comments | « chrome/browser/sync/glue/sync_backend_registrar.cc ('k') | chrome/browser/sync/test_profile_sync_service.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/sync/glue/sync_backend_registrar_unittest.cc
diff --git a/chrome/browser/sync/glue/sync_backend_registrar_unittest.cc b/chrome/browser/sync/glue/sync_backend_registrar_unittest.cc
index 3761067e8f8f7fc0d3ffdad3d8471df9fb4adb13..979f9a5a3f2a5633eb321fd1b0da053415222124 100644
--- a/chrome/browser/sync/glue/sync_backend_registrar_unittest.cc
+++ b/chrome/browser/sync/glue/sync_backend_registrar_unittest.cc
@@ -128,6 +128,7 @@ TEST_F(SyncBackendRegistrarTest, ConfigureDataTypes) {
ExpectRoutingInfo(&registrar, expected_routing_info);
}
ExpectHasProcessorsForTypes(registrar, ModelTypeSet());
+ EXPECT_TRUE(types1.Equals(registrar.GetLastConfiguredTypes()));
// Add and remove.
const ModelTypeSet types2(PREFERENCES, THEMES);
@@ -139,11 +140,13 @@ TEST_F(SyncBackendRegistrarTest, ConfigureDataTypes) {
ExpectRoutingInfo(&registrar, expected_routing_info);
}
ExpectHasProcessorsForTypes(registrar, ModelTypeSet());
+ EXPECT_TRUE(types2.Equals(registrar.GetLastConfiguredTypes()));
// Remove.
EXPECT_TRUE(registrar.ConfigureDataTypes(ModelTypeSet(), types2).Empty());
ExpectRoutingInfo(&registrar, syncer::ModelSafeRoutingInfo());
ExpectHasProcessorsForTypes(registrar, ModelTypeSet());
+ EXPECT_TRUE(ModelTypeSet().Equals(registrar.GetLastConfiguredTypes()));
registrar.OnSyncerShutdownComplete();
registrar.StopOnUIThread();
« no previous file with comments | « chrome/browser/sync/glue/sync_backend_registrar.cc ('k') | chrome/browser/sync/test_profile_sync_service.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698