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

Unified Diff: chrome/browser/chromeos/login/existing_user_controller.cc

Issue 1012083002: Resolve new GAIA flow's infinite loop. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 9 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/chromeos/login/existing_user_controller.cc
diff --git a/chrome/browser/chromeos/login/existing_user_controller.cc b/chrome/browser/chromeos/login/existing_user_controller.cc
index a32ffb68d48f7ebba48f1d22d3bda68c2516951e..aa9615b1e6b1bcb1f88d2ad8ff16260567373635 100644
--- a/chrome/browser/chromeos/login/existing_user_controller.cc
+++ b/chrome/browser/chromeos/login/existing_user_controller.cc
@@ -693,7 +693,10 @@ void ExistingUserController::WhiteListCheckFailed(const std::string& email) {
ShowError(IDS_LOGIN_ERROR_WHITELIST, email);
}
- login_display_->ShowSigninUI(email);
+ if (StartupUtils::IsWebviewSigninEnabled())
+ login_display_->ShowSigninUI("");
Roman Sorokin (ftl) 2015/03/17 17:00:41 Why is call here with an empty string? Could we ju
Ivan Podogov 2015/03/18 07:36:20 This call is made after the Sign-in UI has been ma
+ else
+ login_display_->ShowSigninUI(email);
if (auth_status_consumer_) {
auth_status_consumer_->OnAuthFailure(

Powered by Google App Engine
This is Rietveld 408576698