Make sure to display an errors that occur during the chrome sign in process

chrome/browser/ui/sync/one_click_signin_helper.cc

 // Copyright (c) 2012 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/sync/one_click_signin_helper.h"
 
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/compiler_specific.h"
 #include "base/metrics/histogram.h"
@@ skipping 15 matching lines @@
 #include "chrome/browser/signin/signin_manager_factory.h"
 #include "chrome/browser/signin/signin_names_io_thread.h"
 #include "chrome/browser/sync/profile_sync_service.h"
 #include "chrome/browser/sync/profile_sync_service_factory.h"
 #include "chrome/browser/tab_contents/tab_util.h"
 #include "chrome/browser/ui/browser_finder.h"
 #include "chrome/browser/ui/browser_window.h"
 #include "chrome/browser/ui/sync/one_click_signin_histogram.h"
 #include "chrome/browser/ui/sync/one_click_signin_sync_starter.h"
 #include "chrome/browser/ui/tab_contents/tab_contents.h"
+#include "chrome/common/chrome_notification_types.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/chrome_version_info.h"
 #include "chrome/common/net/url_util.h"
 #include "chrome/common/pref_names.h"
 #include "chrome/common/url_constants.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/page_navigator.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/browser/web_contents_view.h"
 #include "content/public/common/frame_navigate_params.h"
@@ skipping 323 matching lines @@
 
 // static
 void OneClickSigninHelper::AssociateWithRequestForTesting(
     base::SupportsUserData* request,
     const std::string& email) {
   OneClickSigninRequestUserData::AssociateWithRequest(request, email);
 }
 
 // static
 bool OneClickSigninHelper::CanOffer(content::WebContents* web_contents,
+                                    CanOfferFor can_offer_for,
                                     const std::string& email,
-                                    bool check_connected) {
+                                    int* error_message_id) {
   DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::UI));
 
+  *error_message_id = 0;
+
   if (!web_contents)
     return false;
 
   if (web_contents->GetBrowserContext()->IsOffTheRecord())
     return false;
 
   if (!ProfileSyncService::IsSyncEnabled())
     return false;
 
   Profile* profile =
       Profile::FromBrowserContext(web_contents->GetBrowserContext());
   if (!profile)
     return false;
 
-  if (!profile->GetPrefs()->GetBoolean(prefs::kReverseAutologinEnabled))
+  if (can_offer_for == CAN_OFFER_FOR_INTERSTITAL_ONLY &&
+      !profile->GetPrefs()->GetBoolean(prefs::kReverseAutologinEnabled))
     return false;
 
   if (!SigninManager::AreSigninCookiesAllowed(profile))
     return false;
 
-  if (check_connected) {
+  if (!email.empty()) {
     SigninManager* manager =
         SigninManagerFactory::GetForProfile(profile);
     if (!manager)
       return false;
 
-    if (!manager->GetAuthenticatedUsername().empty())
+    if (!manager->GetAuthenticatedUsername().empty()) {
+      *error_message_id = IDS_SYNC_SETUP_ERROR;
       return false;
+    }
 
     // Make sure this username is not prohibited by policy.
-    if (!manager->IsAllowedUsername(email))
+    if (!manager->IsAllowedUsername(email)) {
+      *error_message_id = IDS_SYNC_LOGIN_NAME_PROHIBITED;
       return false;
+    }
 
     // If some profile, not just the current one, is already connected to this
     // account, don't show the infobar.
     if (g_browser_process) {
       ProfileManager* manager = g_browser_process->profile_manager();
       if (manager) {
         string16 email16 = UTF8ToUTF16(email);
         ProfileInfoCache& cache = manager->GetProfileInfoCache();
 
         for (size_t i = 0; i < cache.GetNumberOfProfiles(); ++i) {
-          if (email16 == cache.GetUserNameOfProfileAtIndex(i))
+          if (email16 == cache.GetUserNameOfProfileAtIndex(i)) {
+            *error_message_id = IDS_SYNC_USER_NAME_IN_USE_ERROR;
             return false;
+          }
         }
       }
     }
 
     // If email was already rejected by this profile for one-click sign-in.
-    if (!email.empty()) {
+    if (can_offer_for == CAN_OFFER_FOR_INTERSTITAL_ONLY) {
       const ListValue* rejected_emails = profile->GetPrefs()->GetList(
           prefs::kReverseAutologinRejectedEmailList);
       if (!rejected_emails->empty()) {
         const scoped_ptr<Value> email_value(new base::StringValue(email));
         base::ListValue::const_iterator iter = rejected_emails->Find(
             *email_value);
         if (iter != rejected_emails->end())
           return false;
       }
     }
@@ skipping 217 matching lines @@
     SyncPromoUI::Source source,
     int child_id,
     int route_id) {
   DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::UI));
 
   content::WebContents* web_contents = tab_util::GetWebContentsByID(child_id,
                                                                     route_id);
 
   // TODO(mathp): The appearance of this infobar should be tested using a
   // browser_test.
-  if (!web_contents || !CanOffer(web_contents, email, !email.empty())) {
+  OneClickSigninHelper* helper =
+      OneClickSigninHelper::FromWebContents(web_contents);
+  int error_message_id = 0;
+
+  CanOfferFor can_offer_for =
+      (auto_accept != AUTO_ACCEPT_EXPLICIT &&
+          helper->auto_accept_ != AUTO_ACCEPT_EXPLICIT) ?
+          CAN_OFFER_FOR_INTERSTITAL_ONLY : CAN_OFFER_FOR_ALL;
+
+  if (!web_contents || !CanOffer(web_contents, can_offer_for, email,
+                                 &error_message_id)) {
     VLOG(1) << "OneClickSigninHelper::ShowInfoBarUIThread: not offering";
+    if (helper && helper->error_message_.empty() && error_message_id != 0)
+      helper->error_message_ = l10n_util::GetStringUTF8(error_message_id);
+
     return;
   }
 
   // Save the email in the one-click signin manager.  The manager may
   // not exist if the contents is incognito or if the profile is already
   // connected to a Google account.
-  OneClickSigninHelper* helper =
-      OneClickSigninHelper::FromWebContents(web_contents);
   if (helper) {
     if (!session_index.empty())
       helper->session_index_ = session_index;
 
     if (!email.empty())
       helper->email_ = email;
 
-    helper->auto_accept_ = auto_accept;
-    helper->source_ = source;
+    if (auto_accept != NO_AUTO_ACCEPT) {
+      helper->auto_accept_ = auto_accept;
+      helper->source_ = source;
+    }
   }
 }
 
+void OneClickSigninHelper::RedirectToNTP() {
+  // Redirect to NTP with sign in bubble visible.
+  content::WebContents* contents = web_contents();
+  Profile* profile =
+      Profile::FromBrowserContext(contents->GetBrowserContext());
+  PrefService* pref_service = profile->GetPrefs();
+  pref_service->SetBoolean(prefs::kSyncPromoShowNTPBubble, true);
+  pref_service->SetString(prefs::kSyncPromoErrorMessage, error_message_);

[Andrew T Wilson (Slow), 2012/12/03 14:54:49]

I guess the way this works is once the NTP shows the message, these prefs are
cleared?

[Roger Tawa OOO till Jul 10th, 2012/12/03 22:26:54]

correct.

+
+  contents->GetController().LoadURL(GURL(chrome::kChromeUINewTabURL),
+                                    content::Referrer(),
+                                    content::PAGE_TRANSITION_AUTO_TOPLEVEL,
+                                    std::string());
+
+  error_message_.clear();
+  signin_tracker_.reset();
+}
+
+void OneClickSigninHelper::CleanTransientState() {
+  email_.clear();
+  password_.clear();
+  auto_accept_ = NO_AUTO_ACCEPT;
+  source_ = SyncPromoUI::SOURCE_UNKNOWN;
+}
+
 void OneClickSigninHelper::DidNavigateAnyFrame(
     const content::LoadCommittedDetails& details,
     const content::FrameNavigateParams& params) {
   // We only need to scrape the password for Gaia logins.
   const content::PasswordForm& form = params.password_form;
   if (form.origin.is_valid() &&
       gaia::IsGaiaSignonRealm(GURL(form.signon_realm))) {
     VLOG(1) << "OneClickSigninHelper::DidNavigateAnyFrame: got password";
     password_ = UTF16ToUTF8(params.password_form.password_value);
   }
 }
 
 void OneClickSigninHelper::DidStopLoading(
     content::RenderViewHost* render_view_host) {
+  // If the user left the sign in process, clear all members.
+  // TODO(rogerta): might need to allow some youtube URLs.
+  content::WebContents* contents = web_contents();
+  if (!gaia::IsGaiaSignonRealm(contents->GetURL().GetOrigin())) {
+    CleanTransientState();
+    return;
+  }
+
+  if (!error_message_.empty()) {
+    RedirectToNTP();
+    return;
+  }
+
   if (email_.empty() || password_.empty())
     return;
 
-  content::WebContents* contents = web_contents();
   Browser* browser = chrome::FindBrowserWithWebContents(contents);
   InfoBarTabHelper* infobar_tab_helper =
       InfoBarTabHelper::FromWebContents(contents);
 
   switch (auto_accept_) {
     case AUTO_ACCEPT:
       browser->window()->ShowOneClickSigninBubble(
           base::Bind(&StartSync, browser, auto_accept_, session_index_,
                      email_, password_));
       break;
@@ skipping 25 matching lines @@
   // If this explicit sign in is not from settings page, show the NTP after
   // sign in completes.  In the case of the settings page, it will get closed
   // by SyncSetupHandler.
   if (auto_accept_ == AUTO_ACCEPT_EXPLICIT &&
       source_ != SyncPromoUI::SOURCE_SETTINGS) {
     Profile* profile =
         Profile::FromBrowserContext(contents->GetBrowserContext());
     signin_tracker_.reset(new SigninTracker(profile, this));
   }
 
-  email_.clear();
-  password_.clear();
-  auto_accept_ = NO_AUTO_ACCEPT;
-  source_ = SyncPromoUI::SOURCE_UNKNOWN;
+  CleanTransientState();
 }
 
 void OneClickSigninHelper::GaiaCredentialsValid() {
-  // Redirect to NTP with sign in bubble visible.
-  content::WebContents* contents = web_contents();
-  Profile* profile =
-      Profile::FromBrowserContext(contents->GetBrowserContext());
-  PrefService* pref_service = profile->GetPrefs();
-  pref_service->SetBoolean(prefs::kSyncPromoShowNTPBubble, true);
-
-  contents->GetController().LoadURL(GURL(chrome::kChromeUINewTabURL),
-                                    content::Referrer(),
-                                    content::PAGE_TRANSITION_AUTO_TOPLEVEL,
-                                    std::string());
 }
 
 void OneClickSigninHelper::SigninFailed(const GoogleServiceAuthError& error) {
-  signin_tracker_.reset();
+  if (error_message_.empty() && !error.error_message().empty())
+      error_message_ = error.error_message();
+
+  if (error_message_.empty()) {
+    switch (error.state()) {
+      case GoogleServiceAuthError::NONE:
+        error_message_.clear();
+        break;
+      case GoogleServiceAuthError::SERVICE_UNAVAILABLE:
+        error_message_ = l10n_util::GetStringUTF8(IDS_SYNC_UNRECOVERABLE_ERROR);
+        break;
+      case GoogleServiceAuthError::HOSTED_NOT_ALLOWED:

[Andrew T Wilson (Slow), 2012/12/03 14:54:49]

Remove this line since we just want to fall through to the default case anyway?

[Roger Tawa OOO till Jul 10th, 2012/12/03 22:26:54]

Done.

+      default:
+        error_message_ = l10n_util::GetStringUTF8(IDS_SYNC_ERROR_SIGNING_IN);
+        break;
+    }
+  }
+
+  RedirectToNTP();
 }
 
 void OneClickSigninHelper::SigninSuccess() {
-  signin_tracker_.reset();
+  RedirectToNTP();
 }