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

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

Issue 2388673002: Revert of [Sync] Move //components/sync to the syncer namespace. (patchset #5 id:40001 of https://co (Closed)
Patch Set: Created 4 years, 2 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_registrar.h
diff --git a/components/sync/driver/glue/sync_backend_registrar.h b/components/sync/driver/glue/sync_backend_registrar.h
index 341dceff9e98476e070c0c4fe65dc381549a7975..6879f53fc254a8858e7e789a622426acb0cf7203 100644
--- a/components/sync/driver/glue/sync_backend_registrar.h
+++ b/components/sync/driver/glue/sync_backend_registrar.h
@@ -28,23 +28,29 @@ class MessageLoop;
}
namespace syncer {
+struct UserShare;
+} // namespace syncer
+namespace sync_driver {
class ChangeProcessor;
class SyncClient;
+}
+
+namespace browser_sync {
+
class UIModelWorker;
-struct UserShare;
// A class that keep track of the workers, change processors, and
// routing info for the enabled sync types, and also routes change
// events to the right processors.
-class SyncBackendRegistrar : public SyncManager::ChangeDelegate,
- public WorkerLoopDestructionObserver {
+class SyncBackendRegistrar : public syncer::SyncManager::ChangeDelegate,
+ public syncer::WorkerLoopDestructionObserver {
public:
// |name| is used for debugging. Does not take ownership of |profile|.
// Must be created on the UI thread.
SyncBackendRegistrar(
const std::string& name,
- SyncClient* sync_client,
+ sync_driver::SyncClient* sync_client,
std::unique_ptr<base::Thread> sync_thread,
const scoped_refptr<base::SingleThreadTaskRunner>& ui_thread,
const scoped_refptr<base::SingleThreadTaskRunner>& db_thread,
@@ -67,17 +73,17 @@ class SyncBackendRegistrar : public SyncManager::ChangeDelegate,
// GROUP_NON_BLOCKING model safe group and will be treated differently in
// ModelTypeRegistry. Unlike directory types, non-blocking types always stay
// assigned to GROUP_NON_BLOCKING group.
- void RegisterNonBlockingType(ModelType type);
+ void RegisterNonBlockingType(syncer::ModelType type);
// Informs the SyncBackendRegistrar of the currently enabled set of types.
// These types will be placed in the passive group. This function should be
// called exactly once during startup.
- void SetInitialTypes(ModelTypeSet initial_types);
+ void SetInitialTypes(syncer::ModelTypeSet initial_types);
// Informs SyncBackendRegistrar about non-blocking type loaded from local
// storage. Initial sync was already performed for this type, therefore its
// data shouldn't be downloaded as part of configuration.
- void AddRestoredNonBlockingType(ModelType type);
+ void AddRestoredNonBlockingType(syncer::ModelType type);
// Returns whether or not we are currently syncing encryption keys.
// Must be called on the UI thread.
@@ -88,13 +94,13 @@ class SyncBackendRegistrar : public SyncManager::ChangeDelegate,
// not already there (initially put in the passive group).
// |types_to_remove| and |types_to_add| must be disjoint. Returns
// the set of newly-added types. Must be called on the UI thread.
- ModelTypeSet ConfigureDataTypes(ModelTypeSet types_to_add,
- ModelTypeSet types_to_remove);
+ syncer::ModelTypeSet ConfigureDataTypes(syncer::ModelTypeSet types_to_add,
+ syncer::ModelTypeSet types_to_remove);
// Returns the set of enabled types as of the last configuration. Note that
// this might be different from the current types in the routing info due
// to DeactiveDataType being called separately from ConfigureDataTypes.
- ModelTypeSet GetLastConfiguredTypes() const;
+ syncer::ModelTypeSet GetLastConfiguredTypes() const;
// Must be called from the UI thread. (See destructor comment.)
void RequestWorkerStopOnUIThread();
@@ -102,33 +108,34 @@ class SyncBackendRegistrar : public SyncManager::ChangeDelegate,
// Activates the given data type (which should belong to the given
// group) and starts the given change processor. Must be called
// from |group|'s native thread.
- void ActivateDataType(ModelType type,
- ModelSafeGroup group,
- ChangeProcessor* change_processor,
- UserShare* user_share);
+ void ActivateDataType(syncer::ModelType type,
+ syncer::ModelSafeGroup group,
+ sync_driver::ChangeProcessor* change_processor,
+ syncer::UserShare* user_share);
// Deactivates the given type if necessary. Must be called from the
// UI thread and not |type|'s native thread. Yes, this is
// surprising: see http://crbug.com/92804.
- void DeactivateDataType(ModelType type);
+ void DeactivateDataType(syncer::ModelType type);
// Returns true only between calls to ActivateDataType(type, ...)
// and DeactivateDataType(type). Used only by tests.
- bool IsTypeActivatedForTest(ModelType type) const;
+ bool IsTypeActivatedForTest(syncer::ModelType type) const;
// SyncManager::ChangeDelegate implementation. May be called from
// any thread.
- void OnChangesApplied(ModelType model_type,
- int64_t model_version,
- const BaseTransaction* trans,
- const ImmutableChangeRecordList& changes) override;
- void OnChangesComplete(ModelType model_type) override;
+ void OnChangesApplied(
+ syncer::ModelType model_type,
+ int64_t model_version,
+ const syncer::BaseTransaction* trans,
+ const syncer::ImmutableChangeRecordList& changes) override;
+ void OnChangesComplete(syncer::ModelType model_type) override;
- void GetWorkers(std::vector<scoped_refptr<ModelSafeWorker>>* out);
- void GetModelSafeRoutingInfo(ModelSafeRoutingInfo* out);
+ void GetWorkers(std::vector<scoped_refptr<syncer::ModelSafeWorker>>* out);
+ void GetModelSafeRoutingInfo(syncer::ModelSafeRoutingInfo* out);
- // WorkerLoopDestructionObserver implementation.
- void OnWorkerLoopDestroyed(ModelSafeGroup group) override;
+ // syncer::WorkerLoopDestructionObserver implementation.
+ void OnWorkerLoopDestroyed(syncer::ModelSafeGroup group) override;
// Release ownership of |sync_thread_|. Called when sync is disabled.
std::unique_ptr<base::Thread> ReleaseSyncThread();
@@ -139,45 +146,50 @@ class SyncBackendRegistrar : public SyncManager::ChangeDelegate,
base::Thread* sync_thread();
private:
- typedef std::map<ModelSafeGroup, scoped_refptr<ModelSafeWorker>> WorkerMap;
- typedef std::map<ModelType, ChangeProcessor*> ProcessorMap;
+ typedef std::map<syncer::ModelSafeGroup,
+ scoped_refptr<syncer::ModelSafeWorker>>
+ WorkerMap;
+ typedef std::map<syncer::ModelType, sync_driver::ChangeProcessor*>
+ ProcessorMap;
// Add a worker for |group| to the worker map if one can be created.
- void MaybeAddWorker(ModelSafeGroup group);
+ void MaybeAddWorker(syncer::ModelSafeGroup group);
// Callback after workers unregister from observing destruction of their
// working loops.
- void OnWorkerUnregistrationDone(ModelSafeGroup group);
+ void OnWorkerUnregistrationDone(syncer::ModelSafeGroup group);
- void RemoveWorker(ModelSafeGroup group);
+ void RemoveWorker(syncer::ModelSafeGroup group);
// Returns the change processor for the given model, or NULL if none
// exists. Must be called from |group|'s native thread.
- ChangeProcessor* GetProcessor(ModelType type) const;
+ sync_driver::ChangeProcessor* GetProcessor(syncer::ModelType type) const;
// Must be called with |lock_| held. Simply returns the change
// processor for the given type, if it exists. May be called from
// any thread.
- ChangeProcessor* GetProcessorUnsafe(ModelType type) const;
+ sync_driver::ChangeProcessor* GetProcessorUnsafe(
+ syncer::ModelType type) const;
// Return true if |model_type| lives on the current thread. Must be
// called with |lock_| held. May be called on any thread.
- bool IsCurrentThreadSafeForModel(ModelType model_type) const;
+ bool IsCurrentThreadSafeForModel(syncer::ModelType model_type) const;
// Returns true if the current thread is the native thread for the
// given group (or if it is undeterminable).
- bool IsOnThreadForGroup(ModelType type, ModelSafeGroup group) const;
+ bool IsOnThreadForGroup(syncer::ModelType type,
+ syncer::ModelSafeGroup group) const;
// Returns model safe group that should be assigned to type when it is first
// configured (before activation). Returns GROUP_PASSIVE for directory types
// and GROUP_NON_BLOCKING for non-blocking types.
- ModelSafeGroup GetInitialGroupForType(ModelType type) const;
+ syncer::ModelSafeGroup GetInitialGroupForType(syncer::ModelType type) const;
// Name used for debugging.
const std::string name_;
// A pointer to the sync client.
- SyncClient* const sync_client_;
+ sync_driver::SyncClient* const sync_client_;
// Protects all variables below.
mutable base::Lock lock_;
@@ -189,21 +201,21 @@ class SyncBackendRegistrar : public SyncManager::ChangeDelegate,
// destroyed. Unless a worker is no longer needed because all types
// that get routed to it have been disabled (from syncing). In that
// case, we'll destroy on demand *after* routing any dependent types
- // to GROUP_PASSIVE, so that the syncapi doesn't call into garbage.
+ // to syncer::GROUP_PASSIVE, so that the syncapi doesn't call into garbage.
// If a key is present, it means at least one ModelType that routes
// to that model safe group is being synced.
WorkerMap workers_;
- ModelSafeRoutingInfo routing_info_;
+ syncer::ModelSafeRoutingInfo routing_info_;
// The change processors that handle the different data types.
ProcessorMap processors_;
// The types that were enabled as of the last configuration. Updated on each
// call to ConfigureDataTypes as well as SetInitialTypes.
- ModelTypeSet last_configured_types_;
+ syncer::ModelTypeSet last_configured_types_;
// Parks stopped workers because they may still be referenced by syncer.
- std::vector<scoped_refptr<ModelSafeWorker>> stopped_workers_;
+ std::vector<scoped_refptr<syncer::ModelSafeWorker>> stopped_workers_;
// References to the thread task runners that sync depends on.
const scoped_refptr<base::SingleThreadTaskRunner> ui_thread_;
@@ -218,11 +230,11 @@ class SyncBackendRegistrar : public SyncManager::ChangeDelegate,
// Set of types with non-blocking implementation (as opposed to directory
// based).
- ModelTypeSet non_blocking_types_;
+ syncer::ModelTypeSet non_blocking_types_;
DISALLOW_COPY_AND_ASSIGN(SyncBackendRegistrar);
};
-} // namespace syncer
+} // namespace browser_sync
#endif // COMPONENTS_SYNC_DRIVER_GLUE_SYNC_BACKEND_REGISTRAR_H_
« no previous file with comments | « components/sync/driver/glue/sync_backend_host_mock.cc ('k') | components/sync/driver/glue/sync_backend_registrar.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698