signin: force web signin flow initiated visits to accounts.google.com to their own process.

chrome/browser/ui/webui/sync_promo/sync_promo_ui.cc

Index: chrome/browser/ui/webui/sync_promo/sync_promo_ui.cc
diff --git a/chrome/browser/ui/webui/sync_promo/sync_promo_ui.cc b/chrome/browser/ui/webui/sync_promo/sync_promo_ui.cc
index 4c0920d235cedc0c7b9665202be3ad1614aaa0b5..21359920896a2ddbeb381decdd867286e1036cc2 100644
--- a/chrome/browser/ui/webui/sync_promo/sync_promo_ui.cc
+++ b/chrome/browser/ui/webui/sync_promo/sync_promo_ui.cc
@@ -259,6 +259,16 @@ GURL SyncPromoUI::GetSyncPromoURL(const GURL& next_page,
 }
 
 // static
+bool SyncPromoUI::IsWebBasedSigninFlowURL(const GURL& url) {
+  if (url.SchemeIs(chrome::kChromeSigninScheme))
+    return true;
+
+  return StartsWithASCII(url.spec(),
+                         GaiaUrls::GetInstance()->service_login_url(),
+                         false);

[Roger Tawa OOO till Jul 10th, 2013/02/28 21:13:06]

Nit: I think it might be better to compare using GURL::ReplaceComponents() to
strip off the query part, but up to you if you want to change.

[tim (not reviewing), 2013/03/01 01:53:57]

Yeah I didn't intend on committing as is, but I changed to use GetOrigin and
path() equality.

+}
+
+// static
 GURL SyncPromoUI::GetNextPageURLForSyncPromoURL(const GURL& url) {
   const char* key_name = UseWebBasedSigninFlow() ? kSyncPromoQueryKeyContinue :
       kSyncPromoQueryKeyNextPage;