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

Unified Diff: chrome/browser/ui/webui/chromeos/login/signin_screen_handler.cc

Issue 145073003: Indicate which authentication flow was used in UserContext (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Speculative fix for telemetry. Created 6 years, 11 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
« no previous file with comments | « chrome/browser/ui/webui/chromeos/login/signin_screen_handler.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/webui/chromeos/login/signin_screen_handler.cc
diff --git a/chrome/browser/ui/webui/chromeos/login/signin_screen_handler.cc b/chrome/browser/ui/webui/chromeos/login/signin_screen_handler.cc
index 1091c8a012482784917d98dddecf94874ea6c93e..bee7eb9489d02dccac911bc4597fe9a3cd7d56ff 100644
--- a/chrome/browser/ui/webui/chromeos/login/signin_screen_handler.cc
+++ b/chrome/browser/ui/webui/chromeos/login/signin_screen_handler.cc
@@ -1011,14 +1011,21 @@ void SigninScreenHandler::UpdateAddButtonStatus() {
}
void SigninScreenHandler::HandleCompleteLogin(const std::string& typed_email,
- const std::string& password) {
+ const std::string& password,
+ bool using_saml) {
if (!delegate_)
return;
+
const std::string sanitized_email = gaia::SanitizeEmail(typed_email);
delegate_->SetDisplayEmail(sanitized_email);
- delegate_->CompleteLogin(UserContext(sanitized_email,
- password,
- std::string())); // auth_code
+ delegate_->CompleteLogin(UserContext(
+ sanitized_email,
+ password,
+ std::string(), // auth_code
+ std::string(), // username_hash
+ true, // using_oauth
+ using_saml ? UserContext::AUTH_FLOW_GAIA_WITH_SAML
+ : UserContext::AUTH_FLOW_GAIA_WITHOUT_SAML));
if (test_expects_complete_login_) {
VLOG(2) << "Complete test login for " << typed_email
« no previous file with comments | « chrome/browser/ui/webui/chromeos/login/signin_screen_handler.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698