sync: preliminary support for deferred SyncBackendHost initialization

chrome/browser/sync/profile_sync_service.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_SYNC_PROFILE_SYNC_SERVICE_H_
 #define CHROME_BROWSER_SYNC_PROFILE_SYNC_SERVICE_H_
 
 #include <list>
 #include <string>
 
@@ skipping 363 matching lines @@
   // Returns true if OnPassphraseRequired has been called for decryption and
   // we have an encrypted data type enabled.
   virtual bool IsPassphraseRequiredForDecryption() const;
 
   syncer::PassphraseRequiredReason passphrase_required_reason() const {
     return passphrase_required_reason_;
   }
 
   // Returns a user-friendly string form of last synced time (in minutes).
   virtual string16 GetLastSyncedTimeString() const;
+  std::string GetBackendInitializationStateString() const;

[Nicolas Zea, 2013/02/19 19:01:06]

Newline above this and add comment about what this describes.

[tim (not reviewing), 2013/02/21 03:50:54]

Done.

 
   // Returns true if startup is suppressed (i.e. user has stopped syncing via
   // the google dashboard).
   virtual bool IsStartSuppressed() const;
 
   ProfileSyncComponentsFactory* factory() { return factory_.get(); }
 
   // The profile we are syncing for.
   Profile* profile() const { return profile_; }
 
@@ skipping 216 matching lines @@
       const invalidation::ObjectId& id,
       const std::string& payload);
 
  protected:
   // Used by test classes that derive from ProfileSyncService.
   virtual browser_sync::SyncBackendHost* GetBackendForTest();
 
   // Helper to install and configure a data type manager.
   void ConfigureDataTypeManager();
 
-  // Starts up the backend sync components.
-  virtual void StartUp();
   // Shuts down the backend sync components.
   // |sync_disabled| indicates if syncing is being disabled or not.
   void ShutdownImpl(bool sync_disabled);
 
   // Return SyncCredentials from the TokenService.
   syncer::SyncCredentials GetCredentials();
 
   // Test need to override this to create backends that allow setting up
   // initial conditions, such as populating sync nodes.
   //
@@ skipping 79 matching lines @@
 
   // Sets the last synced time to the current time.
   void UpdateLastSyncedTime();
 
   void NotifyObservers();
 
   void ClearStaleErrors();
 
   void ClearUnrecoverableError();
 
+  enum StartUpDeferred {

[Nicolas Zea, 2013/02/19 19:01:06]

nit: StartUpPolicy seems like a better name for this

[tim (not reviewing), 2013/02/21 03:50:54]

Went with StartUpDeferredOption, because 'start up policy' sounds pretty
ambiguous (i.e I could see us adding a param to StartUp down the road and
'policy' making sense for that too).  Let me know if this is OK with you.

+    STARTUP_DEFER_BACKEND,
+    STARTUP_IMMEDIATE
+  };
+  void StartUp(StartUpDeferred deferred);
+
+  // Starts up the backend sync components.
+  void StartUpSlowBackendComponents();
+
   // About-flags experiment names for datatypes that aren't enabled by default
   // yet.
   static std::string GetExperimentNameForDataType(
       syncer::ModelType data_type);
 
   // Create and register a new datatype controller.
   void RegisterNewDataType(syncer::ModelType data_type);
 
   // Helper method to process SyncConfigureDone after unwinding the stack that
   // originally posted this SyncConfigureDone.
@@ skipping 175 matching lines @@
   syncer::WeakHandle<syncer::DataTypeDebugInfoListener> debug_info_listener_;
 
   DISALLOW_COPY_AND_ASSIGN(ProfileSyncService);
 };
 
 bool ShouldShowActionOnUI(
     const syncer::SyncProtocolError& error);
 
 
 #endif  // CHROME_BROWSER_SYNC_PROFILE_SYNC_SERVICE_H_