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

Unified Diff: components/sync/driver/glue/sync_backend_host_core.h

Issue 2563883006: [Sync] Separate purge types step from sync manager configuration. (Closed)
Patch Set: Created 4 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: components/sync/driver/glue/sync_backend_host_core.h
diff --git a/components/sync/driver/glue/sync_backend_host_core.h b/components/sync/driver/glue/sync_backend_host_core.h
index 641fed6b2989cfb98b0cc31ce777274bce53a481..55e53726c8fb6bf7254f418279cafafb20a5e339 100644
--- a/components/sync/driver/glue/sync_backend_host_core.h
+++ b/components/sync/driver/glue/sync_backend_host_core.h
@@ -82,18 +82,6 @@ struct DoInitializeOptions {
const std::map<ModelType, int64_t> invalidation_versions;
};
-// Helper struct to handle currying params to
-// SyncBackendHostCore::DoConfigureSyncer.
-struct DoConfigureSyncerTypes {
- DoConfigureSyncerTypes();
- DoConfigureSyncerTypes(const DoConfigureSyncerTypes& other);
- ~DoConfigureSyncerTypes();
- ModelTypeSet to_download;
- ModelTypeSet to_purge;
- ModelTypeSet to_journal;
- ModelTypeSet to_unapply;
-};
-
class SyncBackendHostCore
: public base::RefCountedThreadSafe<SyncBackendHostCore>,
public base::trace_event::MemoryDumpProvider,
@@ -205,9 +193,12 @@ class SyncBackendHostCore
void DoDestroySyncManager(ShutdownReason reason);
// Configuration methods that must execute on sync loop.
+ void DoPurgeDisabledTypes(const ModelTypeSet& to_purge,
+ const ModelTypeSet& to_journal,
+ const ModelTypeSet& to_unapply);
void DoConfigureSyncer(
ConfigureReason reason,
- const DoConfigureSyncerTypes& config_types,
+ const ModelTypeSet& to_download,
const ModelSafeRoutingInfo routing_info,
const base::Callback<void(ModelTypeSet, ModelTypeSet)>& ready_task,
const base::Closure& retry_callback);
« no previous file with comments | « components/browser_sync/abstract_profile_sync_service_test.cc ('k') | components/sync/driver/glue/sync_backend_host_core.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698