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

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

Issue 11360259: Sync: Add DeviceInfo's ChangeProcessor (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix sync_client.cc Created 8 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 | « chrome/browser/sync/glue/device_info.cc ('k') | chrome/browser/sync/glue/sync_backend_host.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/sync/glue/sync_backend_host.h
diff --git a/chrome/browser/sync/glue/sync_backend_host.h b/chrome/browser/sync/glue/sync_backend_host.h
index 6d289cd44a9a435b813e6403bc6757901e26713b..b5ced9a37a96904929bd0383419c246759a59166 100644
--- a/chrome/browser/sync/glue/sync_backend_host.h
+++ b/chrome/browser/sync/glue/sync_backend_host.h
@@ -43,10 +43,11 @@ namespace browser_sync {
class ChangeProcessor;
class ChromeSyncNotificationBridge;
-struct Experiments;
class InvalidatorStorage;
class SyncBackendRegistrar;
class SyncPrefs;
+class SyncedDeviceTracker;
+struct Experiments;
// SyncFrontend is the interface used by SyncBackendHost to communicate with
// the entity that created it and, presumably, is interested in sync-related
@@ -284,6 +285,10 @@ class SyncBackendHost : public BackendDataTypeConfigurer {
void GetModelSafeRoutingInfo(syncer::ModelSafeRoutingInfo* out) const;
+ // Fetches the DeviceInfo ChangeProcessor.
+ // We'll keep this test-only until we have non-test clients.
+ virtual SyncedDeviceTracker* GetSyncedDeviceTrackerForTest();
+
protected:
// The types and functions below are protected so that test
// subclasses can use them.
@@ -374,17 +379,18 @@ class SyncBackendHost : public BackendDataTypeConfigurer {
// An enum representing the steps to initializing the SyncBackendHost.
enum InitializationState {
NOT_ATTEMPTED,
- CREATING_SYNC_MANAGER, // We're waiting for the first callback from the
- // sync thread to inform us that the sync manager
- // has been created.
- NOT_INITIALIZED, // Initialization hasn't completed, but we've
- // constructed a SyncManager.
- DOWNLOADING_NIGORI, // The SyncManager is initialized, but
- // we're fetching sync encryption information.
- ASSOCIATING_NIGORI, // The SyncManager is initialized, and we
- // have the sync encryption information, but we
- // have to update the local encryption state.
- INITIALIZED, // Initialization is complete.
+ CREATING_SYNC_MANAGER, // We're waiting for the first callback from the
+ // sync thread to inform us that the sync
+ // manager has been created.
+ NOT_INITIALIZED, // Initialization hasn't completed, but we've
+ // constructed a SyncManager.
+ DOWNLOADING_CONTROL_TYPES, // The SyncManager is initialized, but
+ // we're fetching metadata, such as encryption
+ // information, from the server.
+ PROCESSING_CONTROL_TYPES, // Running init tasks that require metadata to
+ // be available. This includes registering our
+ // device information and refreshing encryption.
+ INITIALIZED, // Initialization is complete.
};
// Checks if we have received a notice to turn on experimental datatypes
@@ -489,6 +495,10 @@ class SyncBackendHost : public BackendDataTypeConfigurer {
// initialization is done yet.
void StopSyncManagerForShutdown(const base::Closure& closure);
+ // Must be called on |frontend_loop_|. |done_callback| is called on
+ // |frontend_loop_|.
+ void InitialProcessControlTypes(const base::Closure& done_callback);
+
base::WeakPtrFactory<SyncBackendHost> weak_ptr_factory_;
// A thread where all the sync operations happen.
« no previous file with comments | « chrome/browser/sync/glue/device_info.cc ('k') | chrome/browser/sync/glue/sync_backend_host.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698