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

Unified Diff: chrome/browser/ui/sync/one_click_signin_helper.cc

Issue 11316299: Enable web-based sign in flow by default. Can use command line argument (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebased Created 8 years 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/sync/one_click_signin_helper.cc
diff --git a/chrome/browser/ui/sync/one_click_signin_helper.cc b/chrome/browser/ui/sync/one_click_signin_helper.cc
index cae3b88cd2e02c55ec776df3d66c2ac7c4899f42..dea3afb904ce8905fce4552ded3f6060cb237aac 100644
--- a/chrome/browser/ui/sync/one_click_signin_helper.cc
+++ b/chrome/browser/ui/sync/one_click_signin_helper.cc
@@ -109,13 +109,6 @@ void StartSync(Browser* browser,
one_click_signin::HISTOGRAM_MAX);
}
-bool UseWebBasedSigninFlow() {
- const bool use_web_based_singin_flow =
- CommandLine::ForCurrentProcess()->HasSwitch(
- switches::kUseWebBasedSigninFlow);
- return use_web_based_singin_flow;
-}
-
// Determines the source of the sign in. Its either one of the known sign in
// access point (first run, NTP, menu, settings) or its an implicit sign in
// via another Google property. In the former case, "service" is also
@@ -664,6 +657,14 @@ void OneClickSigninHelper::ShowInfoBarIfPossible(net::URLRequest* request,
}
// static
+bool OneClickSigninHelper::UseWebBasedSigninFlow() {
+ static const bool use_web_based_singin_flow =
+ !CommandLine::ForCurrentProcess()->HasSwitch(
+ switches::kUseClientLoginSigninFlow);
+ return use_web_based_singin_flow;
+}
+
+// static
void OneClickSigninHelper::ShowInfoBarUIThread(
const std::string& session_index,
const std::string& email,

Powered by Google App Engine
This is Rietveld 408576698