Chromium Code Reviews| 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() { |