Index: chrome/browser/password_manager/chrome_password_manager_client.cc |
diff --git a/chrome/browser/password_manager/chrome_password_manager_client.cc b/chrome/browser/password_manager/chrome_password_manager_client.cc |
index 4e10e36222734b65fa73689df8d319098249b543..d898a3a028c07d2553ae64c7f5a5b696c0f07969 100644 |
--- a/chrome/browser/password_manager/chrome_password_manager_client.cc |
+++ b/chrome/browser/password_manager/chrome_password_manager_client.cc |
@@ -39,6 +39,7 @@ |
#include "content/public/browser/web_contents.h" |
#include "google_apis/gaia/gaia_urls.h" |
#include "net/base/url_util.h" |
+#include "third_party/re2/re2/re2.h" |
#if defined(OS_ANDROID) |
#include "chrome/browser/android/password_authentication_manager.h" |
@@ -480,8 +481,11 @@ bool ChromePasswordManagerClient::IsURLPasswordWebsiteReauth( |
if (!net::GetValueForKeyInQuery(url, "continue", ¶m_value)) |
return false; |
- return GURL(param_value).host() == |
- GURL(chrome::kPasswordManagerAccountDashboardURL).host(); |
+ // All password sites, including test sites, have autofilling disabled. |
+ CR_DEFINE_STATIC_LOCAL(RE2, account_dashboard_pattern, |
+ ("passwords(-([a-z]+\\.corp))?\\.google\\.com")); |
Mike West
2014/09/25 08:26:19
Only google.com? Or do we need to account for othe
Garrett Casto
2014/09/25 22:17:03
Only google.com. This will not be on other TLDs.
|
+ |
+ return RE2::FullMatch(GURL(param_value).host(), account_dashboard_pattern); |
} |
bool ChromePasswordManagerClient::IsTheHotNewBubbleUIEnabled() { |