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

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

Issue 18132003: [Sync] Trigger unapply + getupdates for persistence errors (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix formatting 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
Index: chrome/browser/sync/glue/sync_backend_host.cc
diff --git a/chrome/browser/sync/glue/sync_backend_host.cc b/chrome/browser/sync/glue/sync_backend_host.cc
index eb1b5ca1f531e16030aac6381f75cfaa9870dcfd..870c4d565afae6c358e421fb5a921891e89f132b 100644
--- a/chrome/browser/sync/glue/sync_backend_host.cc
+++ b/chrome/browser/sync/glue/sync_backend_host.cc
@@ -637,9 +637,14 @@ void SyncBackendHost::ConfigureDataTypes(
GetDataTypesInState(CRYPTO, config_state_map);
disabled_types.PutAll(fatal_types);
disabled_types.PutAll(crypto_types);
+ syncer::ModelTypeSet active_types =
+ GetDataTypesInState(CONFIGURE_ACTIVE, config_state_map);
+ syncer::ModelTypeSet clean_first_types =
+ GetDataTypesInState(CONFIGURE_CLEAN, config_state_map);
syncer::ModelTypeSet types_to_download = registrar_->ConfigureDataTypes(
- GetDataTypesInState(CONFIGURE_ACTIVE, config_state_map),
+ syncer::Union(active_types, clean_first_types),
disabled_types);
+ types_to_download.PutAll(clean_first_types);
types_to_download.RemoveAll(syncer::ProxyTypes());
if (!types_to_download.Empty())
types_to_download.Put(syncer::NIGORI);
@@ -699,7 +704,7 @@ void SyncBackendHost::ConfigureDataTypes(
types_to_download,
types_to_purge,
fatal_types,
- crypto_types,
+ syncer::Union(crypto_types, clean_first_types),
inactive_types,
routing_info,
ready_task,

Powered by Google App Engine
This is Rietveld 408576698