 Chromium Code Reviews
 Chromium Code Reviews Issue 1858673002:
  [Sync] Inject startup dependencies into StartupController.  (Closed) 
  Base URL: https://chromium.googlesource.com/chromium/src.git@master
    
  
    Issue 1858673002:
  [Sync] Inject startup dependencies into StartupController.  (Closed) 
  Base URL: https://chromium.googlesource.com/chromium/src.git@master| Index: components/sync_driver/startup_controller.h | 
| diff --git a/components/sync_driver/startup_controller.h b/components/sync_driver/startup_controller.h | 
| index 443d9f9d36bb46ce10e272a5b59db68c47b058e3..0de01857a341ef77832fcef204c6ed0779fdeb87 100644 | 
| --- a/components/sync_driver/startup_controller.h | 
| +++ b/components/sync_driver/startup_controller.h | 
| @@ -10,9 +10,6 @@ | 
| #include "base/time/time.h" | 
| #include "sync/internal_api/public/base/model_type.h" | 
| -class ProfileOAuth2TokenService; | 
| -class SigninManagerWrapper; | 
| - | 
| namespace sync_driver { | 
| class SyncPrefs; | 
| } | 
| @@ -24,9 +21,8 @@ namespace browser_sync { | 
| // to as "the backend"). | 
| class StartupController { | 
| public: | 
| - StartupController(const ProfileOAuth2TokenService* token_service, | 
| - const sync_driver::SyncPrefs* sync_prefs, | 
| - const SigninManagerWrapper* signin, | 
| + StartupController(const sync_driver::SyncPrefs* sync_prefs, | 
| + base::Callback<bool()> can_start, | 
| base::Closure start_backend); | 
| ~StartupController(); | 
| @@ -89,9 +85,7 @@ class StartupController { | 
| const sync_driver::SyncPrefs* sync_prefs_; | 
| - const ProfileOAuth2TokenService* token_service_; | 
| - | 
| - const SigninManagerWrapper* signin_; | 
| + base::Callback<bool()> can_start_; | 
| 
Nicolas Zea
2016/04/04 20:44:17
comment what this is for
 
maxbogue
2016/04/04 22:56:55
Done.
 | 
| // The callback we invoke when it's time to call expensive | 
| // startup routines for the sync backend. |