Fix sign-in error strings.

chrome/browser/ui/views/profiles/profile_chooser_view.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ui/views/profiles/profile_chooser_view.h"
 
 #include "base/prefs/pref_service.h"
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/lifetime/application_lifetime.h"
 #include "chrome/browser/profiles/profile_avatar_icon_util.h"
 #include "chrome/browser/profiles/profile_info_cache.h"
 #include "chrome/browser/profiles/profile_manager.h"
 #include "chrome/browser/profiles/profile_metrics.h"
 #include "chrome/browser/profiles/profile_window.h"
 #include "chrome/browser/profiles/profiles_state.h"
 #include "chrome/browser/signin/profile_oauth2_token_service_factory.h"
 #include "chrome/browser/signin/signin_manager_factory.h"
 #include "chrome/browser/signin/signin_promo.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_commands.h"
 #include "chrome/browser/ui/browser_dialogs.h"
 #include "chrome/browser/ui/chrome_pages.h"
 #include "chrome/browser/ui/singleton_tabs.h"
 #include "chrome/browser/ui/views/profiles/user_manager_view.h"
 #include "chrome/common/pref_names.h"
 #include "chrome/common/url_constants.h"
 #include "components/signin/core/browser/mutable_profile_oauth2_token_service.h"
 #include "components/signin/core/browser/profile_oauth2_token_service.h"
+#include "components/signin/core/browser/signin_error_controller.h"
 #include "components/signin/core/browser/signin_manager.h"
 #include "components/signin/core/common/profile_management_switches.h"
 #include "grit/chromium_strings.h"
 #include "grit/generated_resources.h"
 #include "grit/theme_resources.h"
 #include "third_party/skia/include/core/SkColor.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/base/resource/resource_bundle.h"
 #include "ui/gfx/canvas.h"
 #include "ui/gfx/image/image.h"
@@ skipping 417 matching lines @@
   tutorial_learn_more_link_ = NULL;
   tutorial_enable_new_profile_management_button_ = NULL;
   tutorial_end_preview_link_ = NULL;
   tutorial_send_feedback_button_ = NULL;
   end_preview_and_relaunch_button_ = NULL;
   end_preview_cancel_button_ = NULL;
   remove_account_button_ = NULL;
   account_removal_cancel_button_ = NULL;
   gaia_signin_cancel_button_ = NULL;
   open_other_profile_indexes_map_.clear();
-  current_profile_accounts_map_.clear();
+  delete_account_button_map_.clear();
+  reauth_account_button_map_.clear();
   tutorial_mode_ = TUTORIAL_MODE_NONE;
 }
 
 void ProfileChooserView::Init() {
+  // If view mode is PROFILE_CHOOSER but there is an auth error, force
+  // ACCOUNT_MANAGEMENT mode.
+  if (view_mode_ == BUBBLE_VIEW_MODE_PROFILE_CHOOSER) {
+    SigninErrorController* error =
+        ProfileOAuth2TokenServiceFactory::GetForProfile(browser_->profile())->
+            signin_error_controller();
+    if (error->HasError())
+      view_mode_ = BUBBLE_VIEW_MODE_ACCOUNT_MANAGEMENT;
+  }
+
   ShowView(view_mode_, avatar_menu_.get());
 }
 
 void ProfileChooserView::OnAvatarMenuChanged(
     AvatarMenu* avatar_menu) {
   // Refresh the view with the new menu. We can't just update the local copy
   // as this may have been triggered by a sign out action, in which case
   // the view is being destroyed.
   ShowView(BUBBLE_VIEW_MODE_PROFILE_CHOOSER, avatar_menu);
 }
 
 void ProfileChooserView::OnRefreshTokenAvailable(
     const std::string& account_id) {
   // Refresh the account management view when a new account is added to the
   // profile.
   if (view_mode_ == BUBBLE_VIEW_MODE_ACCOUNT_MANAGEMENT ||
       view_mode_ == BUBBLE_VIEW_MODE_GAIA_SIGNIN ||
-      view_mode_ == BUBBLE_VIEW_MODE_GAIA_ADD_ACCOUNT) {
+      view_mode_ == BUBBLE_VIEW_MODE_GAIA_ADD_ACCOUNT ||
+      view_mode_ == BUBBLE_VIEW_MODE_GAIA_REAUTH) {
     ShowView(BUBBLE_VIEW_MODE_ACCOUNT_MANAGEMENT, avatar_menu_.get());
   }
 }
 
 void ProfileChooserView::OnRefreshTokenRevoked(const std::string& account_id) {
   // Refresh the account management view when an account is removed from the
   // profile.
   if (view_mode_ == BUBBLE_VIEW_MODE_ACCOUNT_MANAGEMENT)
     ShowView(BUBBLE_VIEW_MODE_ACCOUNT_MANAGEMENT, avatar_menu_.get());
 }
@@ skipping 12 matching lines @@
   TutorialMode last_tutorial_mode = tutorial_mode_;
   ResetView();
   RemoveAllChildViews(true);
   view_mode_ = view_to_display;
 
   views::GridLayout* layout;
   views::View* sub_view;
   switch (view_mode_) {
     case BUBBLE_VIEW_MODE_GAIA_SIGNIN:
     case BUBBLE_VIEW_MODE_GAIA_ADD_ACCOUNT:
+    case BUBBLE_VIEW_MODE_GAIA_REAUTH:
       layout = CreateSingleColumnLayout(this, kFixedGaiaViewWidth);
-      sub_view = CreateGaiaSigninView(
-          view_mode_ == BUBBLE_VIEW_MODE_GAIA_ADD_ACCOUNT);
+      sub_view = CreateGaiaSigninView();
       break;
     case BUBBLE_VIEW_MODE_ACCOUNT_REMOVAL:
       layout = CreateSingleColumnLayout(this, kFixedAccountRemovalViewWidth);
       sub_view = CreateAccountRemovalView();
       break;
     case BUBBLE_VIEW_MODE_END_PREVIEW:
       layout = CreateSingleColumnLayout(this, kFixedEndPreviewViewWidth);
       sub_view = CreateEndPreviewView();
       break;
     default:
@@ skipping 86 matching lines @@
     ButtonIndexes::const_iterator profile_match =
         open_other_profile_indexes_map_.find(sender);
     if (profile_match != open_other_profile_indexes_map_.end()) {
       avatar_menu_->SwitchToProfile(
           profile_match->second,
           ui::DispositionFromEventFlags(event.flags()) == NEW_WINDOW,
           ProfileMetrics::SWITCH_PROFILE_ICON);
     } else {
       // This was a profile accounts button.
       AccountButtonIndexes::const_iterator account_match =
-      current_profile_accounts_map_.find(sender);
-      DCHECK(account_match != current_profile_accounts_map_.end());
-      account_id_to_remove_ = account_match->second;
-      ShowView(BUBBLE_VIEW_MODE_ACCOUNT_REMOVAL, avatar_menu_.get());
+          delete_account_button_map_.find(sender);
+      if (account_match != delete_account_button_map_.end()) {
+        account_id_to_remove_ = account_match->second;
+        ShowView(BUBBLE_VIEW_MODE_ACCOUNT_REMOVAL, avatar_menu_.get());
+      } else {
+        account_match = reauth_account_button_map_.find(sender);
+        DCHECK(account_match != reauth_account_button_map_.end());
+        ShowView(BUBBLE_VIEW_MODE_GAIA_REAUTH, avatar_menu_.get());
+      }
     }
   }
 }
 
 void ProfileChooserView::RemoveAccount() {
   DCHECK(!account_id_to_remove_.empty());
   MutableProfileOAuth2TokenService* oauth2_token_service =
       ProfileOAuth2TokenServiceFactory::GetPlatformSpecificForProfile(
       browser_->profile());
   if (oauth2_token_service)
@@ skipping 464 matching lines @@
       profiles::kAvatarBubbleAccountsBackgroundColor));
   views::GridLayout* layout = CreateSingleColumnLayout(view, kFixedMenuWidth);
 
   Profile* profile = browser_->profile();
   std::string primary_account =
       SigninManagerFactory::GetForProfile(profile)->GetAuthenticatedUsername();
   DCHECK(!primary_account.empty());
   std::vector<std::string>accounts =
       profiles::GetSecondaryAccountsForProfile(profile, primary_account);
 
+  // Get state of authentication error, if any.
+  std::string error_account_id;
+  SigninErrorController* error =

[noms (inactive), 2014/05/16 13:37:04]

I think it would be nicer if you had a helper function that returned a boolean.
Something like HasSigninError(Profile*) ?

[Roger Tawa OOO till Jul 10th, 2014/05/16 19:43:49]

Done.

+      ProfileOAuth2TokenServiceFactory::GetForProfile(profile)->
+          signin_error_controller();
+  if (error->HasError())
+    error_account_id = error->error_account_id();
+
   // The primary account should always be listed first.
   // TODO(rogerta): we still need to further differentiate the primary account
   // from the others in the UI, so more work is likely required here:
   // crbug.com/311124.
-  CreateAccountButton(layout, primary_account, true, kFixedMenuWidth);
+  CreateAccountButton(layout, primary_account, true,
+                      error_account_id == primary_account, kFixedMenuWidth);
   for (size_t i = 0; i < accounts.size(); ++i)
-    CreateAccountButton(layout, accounts[i], false, kFixedMenuWidth);
+    CreateAccountButton(layout, accounts[i], false,
+                        error_account_id == accounts[i], kFixedMenuWidth);
   layout->AddPaddingRow(0, views::kRelatedControlVerticalSpacing);
 
   add_account_link_ = CreateLink(l10n_util::GetStringFUTF16(
       IDS_PROFILES_PROFILE_ADD_ACCOUNT_BUTTON, avatar_item.name), this);
   add_account_link_->SetBorder(views::Border::CreateEmptyBorder(
       0, views::kButtonVEdgeMarginNew,
       views::kRelatedControlVerticalSpacing, 0));
   layout->StartRow(1, 0);
   layout->AddView(add_account_link_);
   return view;
 }
 
 void ProfileChooserView::CreateAccountButton(views::GridLayout* layout,
                                              const std::string& account,
                                              bool is_primary_account,
+                                             bool reauth_required,
                                              int width) {
   ui::ResourceBundle* rb = &ui::ResourceBundle::GetSharedInstance();
-  const gfx::ImageSkia* default_image =
+  const gfx::ImageSkia* delete_default_image =
       rb->GetImageNamed(IDR_CLOSE_1).ToImageSkia();
-  int kDeleteButtonWidth = default_image->width();
+  const int kDeleteButtonWidth = delete_default_image->width();
+  const gfx::ImageSkia* warning_default_image = reauth_required ?
+      rb->GetImageNamed(IDR_WARNING).ToImageSkia() : NULL;
+  const int kWarningButtonWidth = reauth_required ?
+      warning_default_image->width() + views::kRelatedButtonHSpacing : 0;
   int available_width = width -
-      kDeleteButtonWidth - views::kButtonHEdgeMarginNew;
+      kDeleteButtonWidth - kWarningButtonWidth - views::kButtonHEdgeMarginNew;
 
   views::LabelButton* email_button = new BackgroundColorHoverButton(
       NULL,
       gfx::ElideEmail(base::UTF8ToUTF16(account),
                       rb->GetFontList(ui::ResourceBundle::BaseFont),
                       available_width),
       gfx::ImageSkia(),
       gfx::ImageSkia());
   layout->StartRow(1, 0);
   layout->AddView(email_button);
 
   // Delete button.
   views::ImageButton* delete_button = new views::ImageButton(this);
   delete_button->SetImageAlignment(views::ImageButton::ALIGN_RIGHT,
                                    views::ImageButton::ALIGN_MIDDLE);
   delete_button->SetImage(views::ImageButton::STATE_NORMAL,
-                          default_image);
+                          delete_default_image);
   delete_button->SetImage(views::ImageButton::STATE_HOVERED,
                           rb->GetImageSkiaNamed(IDR_CLOSE_1_H));
   delete_button->SetImage(views::ImageButton::STATE_PRESSED,
                           rb->GetImageSkiaNamed(IDR_CLOSE_1_P));
   delete_button->SetBounds(
-      available_width, 0, kDeleteButtonWidth, kButtonHeight);
+      available_width + kWarningButtonWidth, 0,
+      kDeleteButtonWidth, kButtonHeight);
 
   email_button->set_notify_enter_exit_on_child(true);
   email_button->AddChildView(delete_button);
 
   // Save the original email address, as the button text could be elided.
-  current_profile_accounts_map_[delete_button] = account;
+  delete_account_button_map_[delete_button] = account;
+
+  // Warning button.
+  if (reauth_required) {
+    views::ImageButton* reauth_button = new views::ImageButton(this);
+    reauth_button->SetImageAlignment(views::ImageButton::ALIGN_LEFT,
+                                     views::ImageButton::ALIGN_MIDDLE);
+    reauth_button->SetImage(views::ImageButton::STATE_NORMAL,
+                            warning_default_image);
+    reauth_button->SetBounds(
+        available_width, 0, kWarningButtonWidth, kButtonHeight);
+
+    email_button->AddChildView(reauth_button);
+    reauth_account_button_map_[reauth_button] = account;
+  }
 }
 
-views::View* ProfileChooserView::CreateGaiaSigninView(
-    bool add_secondary_account) {
+views::View* ProfileChooserView::CreateGaiaSigninView() {
+  GURL url;
+  int message_id;
+
+  switch (view_mode_) {

[noms (inactive), 2014/05/16 13:37:04]

Nice! <3

[Roger Tawa OOO till Jul 10th, 2014/05/16 19:43:49]

Done.

+    case BUBBLE_VIEW_MODE_GAIA_SIGNIN:
+      url = signin::GetPromoURL(signin::SOURCE_AVATAR_BUBBLE_SIGN_IN,
+                                false /* auto_close */,
+                                true /* is_constrained */);
+      message_id = IDS_PROFILES_GAIA_SIGNIN_TITLE;
+      break;
+    case BUBBLE_VIEW_MODE_GAIA_ADD_ACCOUNT:
+      url = signin::GetPromoURL(signin::SOURCE_AVATAR_BUBBLE_ADD_ACCOUNT,
+                                false /* auto_close */,
+                                true /* is_constrained */);
+      message_id = IDS_PROFILES_GAIA_ADD_ACCOUNT_TITLE;
+      break;
+    case BUBBLE_VIEW_MODE_GAIA_REAUTH: {
+      SigninErrorController* error =
+          ProfileOAuth2TokenServiceFactory::GetForProfile(browser_->profile())->
+              signin_error_controller();
+      DCHECK(error->HasError());
+      url = signin::GetReauthURL(browser_->profile(), error->error_username());
+      message_id = IDS_PROFILES_GAIA_REAUTH_TITLE;
+      break;
+    }
+  }
+
   // Adds Gaia signin webview
   Profile* profile = browser_->profile();
   views::WebView* web_view = new views::WebView(profile);
-  signin::Source source = add_secondary_account ?
-      signin::SOURCE_AVATAR_BUBBLE_ADD_ACCOUNT :
-      signin::SOURCE_AVATAR_BUBBLE_SIGN_IN;
-  GURL url(signin::GetPromoURL(
-      source, false /* auto_close */, true /* is_constrained */));
   web_view->LoadInitialURL(url);
   web_view->SetPreferredSize(
       gfx::Size(kFixedGaiaViewWidth, kFixedGaiaViewHeight));
 
-  TitleCard* title_card = new TitleCard(
-      add_secondary_account ? IDS_PROFILES_GAIA_ADD_ACCOUNT_TITLE :
-                              IDS_PROFILES_GAIA_SIGNIN_TITLE,
-      this, &gaia_signin_cancel_button_);
+  TitleCard* title_card = new TitleCard(message_id, this,
+                                        &gaia_signin_cancel_button_);
   return TitleCard::AddPaddedTitleCard(
       web_view, title_card, kFixedGaiaViewWidth);
 }
 
 views::View* ProfileChooserView::CreateAccountRemovalView() {
   views::View* view = new views::View();
   views::GridLayout* layout = CreateSingleColumnLayout(
       view, kFixedAccountRemovalViewWidth - 2 * views::kButtonHEdgeMarginNew);
   layout->SetInsets(0,
                     views::kButtonHEdgeMarginNew,
@@ skipping 92 matching lines @@
   layout->StartRowWithPadding(
       1, 0, 0, views::kUnrelatedControlVerticalSpacing);
   layout->AddView(end_preview_and_relaunch_button_);
 
   TitleCard* title_card = new TitleCard(
       IDS_PROFILES_END_PREVIEW, this, &end_preview_cancel_button_);
   return TitleCard::AddPaddedTitleCard(
       view, title_card, kFixedAccountRemovalViewWidth);
 }