| Index: chrome/browser/ui/sync/one_click_signin_sync_starter.cc
|
| diff --git a/chrome/browser/ui/sync/one_click_signin_sync_starter.cc b/chrome/browser/ui/sync/one_click_signin_sync_starter.cc
|
| index 61e6906612f8d85fedea7328cdf720a4e5eab0fb..3d11b70c73cd6fb3e1acd7ef1672dc5e77e72fa0 100644
|
| --- a/chrome/browser/ui/sync/one_click_signin_sync_starter.cc
|
| +++ b/chrome/browser/ui/sync/one_click_signin_sync_starter.cc
|
| @@ -39,6 +39,7 @@
|
| #include "components/browser_sync/browser/profile_sync_service.h"
|
| #include "components/signin/core/browser/signin_manager.h"
|
| #include "components/signin/core/browser/signin_metrics.h"
|
| +#include "components/signin/core/common/profile_management_switches.h"
|
| #include "components/sync_driver/sync_prefs.h"
|
| #include "net/url_request/url_request_context_getter.h"
|
| #include "ui/base/l10n/l10n_util.h"
|
| @@ -391,14 +392,20 @@ void OneClickSigninSyncStarter::UntrustedSigninConfirmed(
|
| }
|
|
|
| void OneClickSigninSyncStarter::OnSyncConfirmationUIClosed(
|
| - bool configure_sync_first) {
|
| - if (configure_sync_first) {
|
| - chrome::ShowSettingsSubPage(browser_, chrome::kSyncSetupSubPage);
|
| - } else {
|
| - ProfileSyncService* profile_sync_service = GetProfileSyncService();
|
| - if (profile_sync_service)
|
| - profile_sync_service->SetSyncSetupCompleted();
|
| - FinishProfileSyncServiceSetup();
|
| + LoginUIService::SyncConfirmationUIClosedResults results) {
|
| + switch (results) {
|
| + case LoginUIService::CONFIGURE_SYNC_FIRST:
|
| + chrome::ShowSettingsSubPage(browser_, chrome::kSyncSetupSubPage);
|
| + break;
|
| + case LoginUIService::SYNC_WITH_DEFAULT_SETTINGS: {
|
| + ProfileSyncService* profile_sync_service = GetProfileSyncService();
|
| + if (profile_sync_service)
|
| + profile_sync_service->SetSyncSetupCompleted();
|
| + FinishProfileSyncServiceSetup();
|
| + break;
|
| + }
|
| + case LoginUIService::ABORT_SIGNIN:
|
| + break;
|
| }
|
|
|
| delete this;
|
| @@ -458,8 +465,12 @@ void OneClickSigninSyncStarter::AccountAddedToCookie(
|
| break;
|
| }
|
| case CONFIRM_SYNC_SETTINGS_FIRST:
|
| - // Blocks sync until the sync settings confirmation UI is closed.
|
| - DisplayFinalConfirmationBubble(base::string16());
|
| + if (switches::UsePasswordSeparatedSigninFlow()) {
|
| + DisplayModalSyncConfirmationWindow();
|
| + } else {
|
| + // Blocks sync until the sync settings confirmation UI is closed.
|
| + DisplayFinalConfirmationBubble(base::string16());
|
| + }
|
| return;
|
| case CONFIGURE_SYNC_FIRST:
|
| ShowSettingsPage(true); // Show sync config UI.
|
| @@ -488,6 +499,11 @@ void OneClickSigninSyncStarter::DisplayFinalConfirmationBubble(
|
| DisplayLoginResult(browser_, custom_message);
|
| }
|
|
|
| +void OneClickSigninSyncStarter::DisplayModalSyncConfirmationWindow() {
|
| + browser_ = EnsureBrowser(browser_, profile_, desktop_type_);
|
| + browser_->window()->ShowModalSyncConfirmationWindow();
|
| +}
|
| +
|
| // static
|
| Browser* OneClickSigninSyncStarter::EnsureBrowser(
|
| Browser* browser,
|
|
|