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

Unified Diff: chrome/browser/ui/webui/inline_login_ui.cc

Issue 66403006: Use embedded sign in UI for inline sign in flow (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 1 month 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/ui/webui/inline_login_ui.cc
diff --git a/chrome/browser/ui/webui/inline_login_ui.cc b/chrome/browser/ui/webui/inline_login_ui.cc
index cc7be52394a57634b4a1254aafe9ac5ff24d70c3..6d6c5682f7cd629b8d28714086e789cf59e6ac39 100644
--- a/chrome/browser/ui/webui/inline_login_ui.cc
+++ b/chrome/browser/ui/webui/inline_login_ui.cc
@@ -151,17 +151,17 @@ class InlineLoginUIHandler : public content::WebUIMessageHandler {
const GURL& current_url = web_ui()->GetWebContents()->GetURL();
signin::Source source = signin::GetSourceForPromoURL(current_url);
- // TODO(guohui): switch to the embedded gaia endpoint for avatar flows
- // when available.
DCHECK(source != signin::SOURCE_UNKNOWN);
- if (source != signin::SOURCE_AVATAR_BUBBLE_ADD_ACCOUNT &&
- source != signin::SOURCE_AVATAR_BUBBLE_SIGN_IN) {
- params.SetString("service", "chromiumsync");
- base::StringAppendF(
- &encoded_continue_params, "&%s=%d", "source",
- static_cast<int>(source));
+ if (source == signin::SOURCE_AVATAR_BUBBLE_ADD_ACCOUNT ||
+ source == signin::SOURCE_AVATAR_BUBBLE_SIGN_IN) {
+ params.SetString("gaiaPath", gaiaUrls->embedded_signin_url().path());
}
+ params.SetString("service", "chromiumsync");
+ base::StringAppendF(
+ &encoded_continue_params, "&%s=%d", "source",
+ static_cast<int>(source));
+
params.SetString("continueUrl",
gaiaUrls->client_login_to_oauth2_url().Resolve(
encoded_continue_params).spec());

Powered by Google App Engine
This is Rietveld 408576698