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

Unified Diff: chrome/browser/ui/webui/settings/people_handler.cc

Issue 2066493003: Revert of Sync: Support multiple setup UIs. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 6 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: chrome/browser/ui/webui/settings/people_handler.cc
diff --git a/chrome/browser/ui/webui/settings/people_handler.cc b/chrome/browser/ui/webui/settings/people_handler.cc
index feba9683bf5f76df5b8db64449d4e9a89cd226c9..a32d344dea3a5fb52a5cf0988ad1b163c3ef9a24 100644
--- a/chrome/browser/ui/webui/settings/people_handler.cc
+++ b/chrome/browser/ui/webui/settings/people_handler.cc
@@ -304,7 +304,7 @@
ProfileSyncService* service = GetSyncService();
if (service)
- sync_blocker_ = service->GetSetupInProgressHandle();
+ service->SetSetupInProgress(true);
return true;
}
@@ -614,7 +614,8 @@
// Alert the sync service anytime the sync setup dialog is closed. This can
// happen due to the user clicking the OK or Cancel button, or due to the
// dialog being closed by virtue of sync being disabled in the background.
- sync_blocker_.reset();
+ if (sync_service)
+ sync_service->SetSetupInProgress(false);
configuring_sync_ = false;
}
@@ -913,7 +914,7 @@
// We're done configuring, so notify ProfileSyncService that it is OK to
// start syncing.
- sync_blocker_.reset();
+ service->SetSetupInProgress(false);
service->SetFirstSetupComplete();
}
« no previous file with comments | « chrome/browser/ui/webui/settings/people_handler.h ('k') | chrome/browser/ui/webui/settings/people_handler_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698