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

Side by Side Diff: chrome/browser/ui/views/profiles/profile_chooser_view.cc

Issue 2920853004: [sync] Display an error when sync settings aren't confirmed (Closed)
Patch Set: initialize button to nullptr Created 3 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/ui/views/profiles/profile_chooser_view.h" 5 #include "chrome/browser/ui/views/profiles/profile_chooser_view.h"
6 6
7 #include "base/macros.h" 7 #include "base/macros.h"
8 #include "base/metrics/user_metrics.h" 8 #include "base/metrics/user_metrics.h"
9 #include "base/strings/utf_string_conversions.h" 9 #include "base/strings/utf_string_conversions.h"
10 #include "chrome/app/vector_icons/vector_icons.h" 10 #include "chrome/app/vector_icons/vector_icons.h"
(...skipping 558 matching lines...) Expand 10 before | Expand all | Expand 10 after
569 if (oauth2_token_service) 569 if (oauth2_token_service)
570 oauth2_token_service->RemoveObserver(this); 570 oauth2_token_service->RemoveObserver(this);
571 } 571 }
572 572
573 void ProfileChooserView::ResetView() { 573 void ProfileChooserView::ResetView() {
574 open_other_profile_indexes_map_.clear(); 574 open_other_profile_indexes_map_.clear();
575 delete_account_button_map_.clear(); 575 delete_account_button_map_.clear();
576 reauth_account_button_map_.clear(); 576 reauth_account_button_map_.clear();
577 sync_error_signin_button_ = nullptr; 577 sync_error_signin_button_ = nullptr;
578 sync_error_passphrase_button_ = nullptr; 578 sync_error_passphrase_button_ = nullptr;
579 sync_error_settings_unconfirmed_button_ = nullptr;
579 sync_error_upgrade_button_ = nullptr; 580 sync_error_upgrade_button_ = nullptr;
580 sync_error_signin_again_button_ = nullptr; 581 sync_error_signin_again_button_ = nullptr;
581 sync_error_signout_button_ = nullptr; 582 sync_error_signout_button_ = nullptr;
582 manage_accounts_link_ = nullptr; 583 manage_accounts_link_ = nullptr;
583 manage_accounts_button_ = nullptr; 584 manage_accounts_button_ = nullptr;
584 signin_current_profile_button_ = nullptr; 585 signin_current_profile_button_ = nullptr;
585 auth_error_email_button_ = nullptr; 586 auth_error_email_button_ = nullptr;
586 current_profile_card_ = nullptr; 587 current_profile_card_ = nullptr;
587 first_profile_button_ = nullptr; 588 first_profile_button_ = nullptr;
588 guest_profile_button_ = nullptr; 589 guest_profile_button_ = nullptr;
(...skipping 216 matching lines...) Expand 10 before | Expand all | Expand 10 after
805 chrome::NewIncognitoWindow(browser_); 806 chrome::NewIncognitoWindow(browser_);
806 PostActionPerformed(ProfileMetrics::PROFILE_DESKTOP_MENU_GO_INCOGNITO); 807 PostActionPerformed(ProfileMetrics::PROFILE_DESKTOP_MENU_GO_INCOGNITO);
807 } else if (sender == lock_button_) { 808 } else if (sender == lock_button_) {
808 profiles::LockProfile(browser_->profile()); 809 profiles::LockProfile(browser_->profile());
809 PostActionPerformed(ProfileMetrics::PROFILE_DESKTOP_MENU_LOCK); 810 PostActionPerformed(ProfileMetrics::PROFILE_DESKTOP_MENU_LOCK);
810 } else if (sender == close_all_windows_button_) { 811 } else if (sender == close_all_windows_button_) {
811 profiles::CloseProfileWindows(browser_->profile()); 812 profiles::CloseProfileWindows(browser_->profile());
812 } else if (sender == auth_error_email_button_ || 813 } else if (sender == auth_error_email_button_ ||
813 sender == sync_error_signin_button_) { 814 sender == sync_error_signin_button_) {
814 ShowViewFromMode(profiles::BUBBLE_VIEW_MODE_GAIA_REAUTH); 815 ShowViewFromMode(profiles::BUBBLE_VIEW_MODE_GAIA_REAUTH);
815 } else if (sender == sync_error_passphrase_button_) { 816 } else if (sender == sync_error_passphrase_button_ ||
817 sender == sync_error_settings_unconfirmed_button_) {
816 chrome::ShowSettingsSubPage(browser_, chrome::kSyncSetupSubPage); 818 chrome::ShowSettingsSubPage(browser_, chrome::kSyncSetupSubPage);
817 } else if (sender == sync_error_upgrade_button_) { 819 } else if (sender == sync_error_upgrade_button_) {
818 chrome::OpenUpdateChromeDialog(browser_); 820 chrome::OpenUpdateChromeDialog(browser_);
819 } else if (sender == sync_error_signin_again_button_) { 821 } else if (sender == sync_error_signin_again_button_) {
820 if (ProfileSyncServiceFactory::GetForProfile(browser_->profile())) 822 if (ProfileSyncServiceFactory::GetForProfile(browser_->profile()))
821 browser_sync::ProfileSyncService::SyncEvent( 823 browser_sync::ProfileSyncService::SyncEvent(
822 browser_sync::ProfileSyncService::STOP_FROM_OPTIONS); 824 browser_sync::ProfileSyncService::STOP_FROM_OPTIONS);
823 SigninManagerFactory::GetForProfile(browser_->profile()) 825 SigninManagerFactory::GetForProfile(browser_->profile())
824 ->SignOut(signin_metrics::USER_CLICKED_SIGNOUT_SETTINGS, 826 ->SignOut(signin_metrics::USER_CLICKED_SIGNOUT_SETTINGS,
825 signin_metrics::SignoutDelete::IGNORE_METRIC); 827 signin_metrics::SignoutDelete::IGNORE_METRIC);
(...skipping 160 matching lines...) Expand 10 before | Expand all | Expand 10 after
986 if (option_buttons_view) { 988 if (option_buttons_view) {
987 layout->StartRow(0, 0); 989 layout->StartRow(0, 0);
988 layout->AddView(option_buttons_view); 990 layout->AddView(option_buttons_view);
989 } 991 }
990 return view; 992 return view;
991 } 993 }
992 994
993 views::View* ProfileChooserView::CreateSyncErrorViewIfNeeded() { 995 views::View* ProfileChooserView::CreateSyncErrorViewIfNeeded() {
994 int content_string_id, button_string_id; 996 int content_string_id, button_string_id;
995 views::LabelButton** button_out = nullptr; 997 views::LabelButton** button_out = nullptr;
998 SigninManagerBase* signin_manager =
999 SigninManagerFactory::GetForProfile(browser_->profile());
996 sync_ui_util::AvatarSyncErrorType error = 1000 sync_ui_util::AvatarSyncErrorType error =
997 sync_ui_util::GetMessagesForAvatarSyncError( 1001 sync_ui_util::GetMessagesForAvatarSyncError(
998 browser_->profile(), &content_string_id, &button_string_id); 1002 browser_->profile(), *signin_manager, &content_string_id,
1003 &button_string_id);
999 switch (error) { 1004 switch (error) {
1000 case sync_ui_util::MANAGED_USER_UNRECOVERABLE_ERROR: 1005 case sync_ui_util::MANAGED_USER_UNRECOVERABLE_ERROR:
1001 button_out = &sync_error_signout_button_; 1006 button_out = &sync_error_signout_button_;
1002 break; 1007 break;
1003 case sync_ui_util::UNRECOVERABLE_ERROR: 1008 case sync_ui_util::UNRECOVERABLE_ERROR:
1004 button_out = &sync_error_signin_again_button_; 1009 button_out = &sync_error_signin_again_button_;
1005 break; 1010 break;
1006 case sync_ui_util::SUPERVISED_USER_AUTH_ERROR: 1011 case sync_ui_util::SUPERVISED_USER_AUTH_ERROR:
1007 button_out = nullptr; 1012 button_out = nullptr;
1008 break; 1013 break;
1009 case sync_ui_util::AUTH_ERROR: 1014 case sync_ui_util::AUTH_ERROR:
1010 button_out = &sync_error_signin_button_; 1015 button_out = &sync_error_signin_button_;
1011 break; 1016 break;
1012 case sync_ui_util::UPGRADE_CLIENT_ERROR: 1017 case sync_ui_util::UPGRADE_CLIENT_ERROR:
1013 button_out = &sync_error_upgrade_button_; 1018 button_out = &sync_error_upgrade_button_;
1014 break; 1019 break;
1015 case sync_ui_util::PASSPHRASE_ERROR: 1020 case sync_ui_util::PASSPHRASE_ERROR:
1016 button_out = &sync_error_passphrase_button_; 1021 button_out = &sync_error_passphrase_button_;
1017 break; 1022 break;
1023 case sync_ui_util::SETTINGS_UNCONFIRMED_ERROR:
1024 button_out = &sync_error_settings_unconfirmed_button_;
1025 break;
1018 case sync_ui_util::NO_SYNC_ERROR: 1026 case sync_ui_util::NO_SYNC_ERROR:
1019 return nullptr; 1027 return nullptr;
1020 default: 1028 default:
1021 NOTREACHED(); 1029 NOTREACHED();
1022 } 1030 }
1023 1031
1024 ChromeLayoutProvider* provider = ChromeLayoutProvider::Get(); 1032 ChromeLayoutProvider* provider = ChromeLayoutProvider::Get();
1025 1033
1026 // Sets an overall horizontal layout. 1034 // Sets an overall horizontal layout.
1027 views::View* view = new views::View(); 1035 views::View* view = new views::View();
(...skipping 597 matching lines...) Expand 10 before | Expand all | Expand 10 after
1625 IncognitoModePrefs::GetAvailability(browser_->profile()->GetPrefs()) != 1633 IncognitoModePrefs::GetAvailability(browser_->profile()->GetPrefs()) !=
1626 IncognitoModePrefs::DISABLED; 1634 IncognitoModePrefs::DISABLED;
1627 return incognito_available && !browser_->profile()->IsGuestSession(); 1635 return incognito_available && !browser_->profile()->IsGuestSession();
1628 } 1636 }
1629 1637
1630 void ProfileChooserView::PostActionPerformed( 1638 void ProfileChooserView::PostActionPerformed(
1631 ProfileMetrics::ProfileDesktopMenu action_performed) { 1639 ProfileMetrics::ProfileDesktopMenu action_performed) {
1632 ProfileMetrics::LogProfileDesktopMenu(action_performed, gaia_service_type_); 1640 ProfileMetrics::LogProfileDesktopMenu(action_performed, gaia_service_type_);
1633 gaia_service_type_ = signin::GAIA_SERVICE_TYPE_NONE; 1641 gaia_service_type_ = signin::GAIA_SERVICE_TYPE_NONE;
1634 } 1642 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698