Index: components/browser_sync/browser/profile_sync_service.h |
diff --git a/components/browser_sync/browser/profile_sync_service.h b/components/browser_sync/browser/profile_sync_service.h |
index 21570498856c9d50b3f80d3146dde40dcaa387c1..3fecc3197fa8b306fb396ec8b051385d7a41aae3 100644 |
--- a/components/browser_sync/browser/profile_sync_service.h |
+++ b/components/browser_sync/browser/profile_sync_service.h |
@@ -165,16 +165,14 @@ |
// that control the initial sync download: |
// |
// * SetFirstSetupComplete() |
-// * GetSetupInProgressHandle() |
+// * SetSetupInProgress() |
// |
// SetFirstSetupComplete() should be called once the user has finished setting |
-// up sync at least once on their account. GetSetupInProgressHandle() should |
-// be called while the user is actively configuring their account. The handle |
-// should be deleted once configuration is complete. |
-// |
-// Once first setup has completed and there are no outstanding |
-// setup-in-progress handles, CanConfigureDataTypes() will return true and |
-// datatype configuration can begin. |
+// up sync at least once on their account. SetSetupInProgress(true) should be |
+// called while the user is actively configuring their account, and then |
+// SetSetupInProgress(false) should be called when configuration is complete. |
+// Once both these conditions have been met, CanConfigureDataTypes() will |
+// return true and datatype configuration can begin. |
class ProfileSyncService : public sync_driver::SyncService, |
public sync_driver::SyncFrontend, |
public sync_driver::SyncPrefObserver, |
@@ -282,8 +280,7 @@ |
syncer::ModelTypeSet chosen_types) override; |
void SetFirstSetupComplete() override; |
bool IsFirstSetupInProgress() const override; |
- std::unique_ptr<sync_driver::SyncSetupInProgressHandle> |
- GetSetupInProgressHandle() override; |
+ void SetSetupInProgress(bool setup_in_progress) override; |
bool IsSetupInProgress() const override; |
bool ConfigurationDone() const override; |
const GoogleServiceAuthError& GetAuthError() const override; |
@@ -805,9 +802,6 @@ |
// True if setup has been completed at least once and is not in progress. |
bool CanConfigureDataTypes() const; |
- // Called when a SetupInProgressHandle issued by this instance is destroyed. |
- virtual void OnSetupInProgressHandleDestroyed(); |
- |
// This profile's SyncClient, which abstracts away non-Sync dependencies and |
// the Sync API component factory. |
std::unique_ptr<sync_driver::SyncClient> sync_client_; |
@@ -849,10 +843,6 @@ |
// Indicates if this is the first time sync is being configured. This value |
// is equal to !IsFirstSetupComplete() at the time of OnBackendInitialized(). |
bool is_first_time_sync_configure_; |
- |
- // Number of UIs currently configuring the Sync service. When this number |
- // is decremented back to zero, Sync setup is marked no longer in progress. |
- int outstanding_setup_in_progress_handles_ = 0; |
// List of available data type controllers. |
sync_driver::DataTypeController::TypeMap data_type_controllers_; |