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

Unified Diff: components/sync_driver/startup_controller.cc

Issue 1789433002: [Sync] Start up immediately when entering the sync setup UI. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@setup
Patch Set: Created 4 years, 9 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/startup_controller.cc
diff --git a/components/sync_driver/startup_controller.cc b/components/sync_driver/startup_controller.cc
index a8ae9419bbc1fda307597a0d0987f2d615851e88..631cdcce3466a2e60ab2bad542f0b951427be415 100644
--- a/components/sync_driver/startup_controller.cc
+++ b/components/sync_driver/startup_controller.cc
@@ -78,8 +78,11 @@ void StartupController::Reset(const syncer::ModelTypeSet registered_types) {
registered_types_ = registered_types;
}
-void StartupController::set_setup_in_progress(bool in_progress) {
- setup_in_progress_ = in_progress;
+void StartupController::SetSetupInProgress(bool setup_in_progress) {
+ setup_in_progress_ = setup_in_progress;
+ if (setup_in_progress_) {
+ TryStart();
Nicolas Zea 2016/03/14 22:42:01 What happens if Chrome is already started? Can thi
maxbogue 2016/03/15 17:41:01 Assuming you meant s/Chrome/sync, no, it will not
Nicolas Zea 2016/03/15 18:16:14 Acknowledged.
+ }
}
bool StartupController::StartUp(StartUpDeferredOption deferred_option) {

Powered by Google App Engine
This is Rietveld 408576698