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

Unified Diff: components/sync/engine_impl/sync_manager_impl.cc

Issue 2500223002: [Sync] Let ModelTypeRegistry filter out initial download USS types. (Closed)
Patch Set: Address comment. 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 | « components/sync/engine_impl/model_type_registry.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/sync/engine_impl/sync_manager_impl.cc
diff --git a/components/sync/engine_impl/sync_manager_impl.cc b/components/sync/engine_impl/sync_manager_impl.cc
index 7a3466d019f9677252cc1452e80be7f5d712afa6..471bafc175996efcb08fc4a83b808b5289e0e62e 100644
--- a/components/sync/engine_impl/sync_manager_impl.cc
+++ b/components/sync/engine_impl/sync_manager_impl.cc
@@ -194,6 +194,10 @@ void SyncManagerImpl::ConfigureSyncer(
DCHECK(!ready_task.is_null());
DCHECK(initialized_);
+ // Don't download non-blocking types that have already completed initial sync.
+ to_download.RemoveAll(
+ model_type_registry_->GetInitialSyncDoneNonBlockingTypes());
+
DVLOG(1) << "Configuring -"
<< "\n\t"
<< "current types: "
« no previous file with comments | « components/sync/engine_impl/model_type_registry.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698