| Index: chrome/browser/ui/webui/signin/login_ui_service.cc
|
| diff --git a/chrome/browser/ui/webui/signin/login_ui_service.cc b/chrome/browser/ui/webui/signin/login_ui_service.cc
|
| index 713f62c840795a243d3e64c9ffc2e2a20e0c38a5..926c6e02a57b73620a577591316ed5cb1db8fcbc 100644
|
| --- a/chrome/browser/ui/webui/signin/login_ui_service.cc
|
| +++ b/chrome/browser/ui/webui/signin/login_ui_service.cc
|
| @@ -75,24 +75,15 @@ void LoginUIService::ShowLoginPopup() {
|
|
|
| void LoginUIService::DisplayLoginResult(Browser* browser,
|
| const base::string16& message) {
|
| - last_login_result_ = message;
|
| - if (switches::IsNewAvatarMenu()) {
|
| - browser->window()->ShowAvatarBubbleFromAvatarButton(
|
| - message.empty() ? BrowserWindow::AVATAR_BUBBLE_MODE_CONFIRM_SIGNIN :
|
| - BrowserWindow::AVATAR_BUBBLE_MODE_SHOW_ERROR,
|
| - signin::ManageAccountsParams());
|
| - } else {
|
| -#if defined(ENABLE_ONE_CLICK_SIGNIN)
|
| - browser->window()->ShowOneClickSigninBubble(
|
| - BrowserWindow::ONE_CLICK_SIGNIN_BUBBLE_TYPE_BUBBLE,
|
| - base::string16(), /* no SAML email */
|
| - message,
|
| - // This callback is never invoked.
|
| - // TODO(rogerta): Separate out the bubble API so we don't have to pass
|
| - // ignored |email| and |callback| params.
|
| - BrowserWindow::StartSyncCallback());
|
| +#if defined(OS_CHROMEOS)
|
| + // ChromeOS doesn't have the avatar bubble so it never calls this function.
|
| + NOTREACHED();
|
| #endif
|
| - }
|
| + last_login_result_ = message;
|
| + browser->window()->ShowAvatarBubbleFromAvatarButton(
|
| + message.empty() ? BrowserWindow::AVATAR_BUBBLE_MODE_CONFIRM_SIGNIN :
|
| + BrowserWindow::AVATAR_BUBBLE_MODE_SHOW_ERROR,
|
| + signin::ManageAccountsParams());
|
| }
|
|
|
| const base::string16& LoginUIService::GetLastLoginResult() {
|
|
|