Index: components/password_manager/core/browser/password_form_manager.cc |
diff --git a/components/password_manager/core/browser/password_form_manager.cc b/components/password_manager/core/browser/password_form_manager.cc |
index 529fe5f9471f088d1e0fe68ff495dde7a9e3cc7b..60d71423c43d7cb7bb9f28bfda7f4a2d6b617ad0 100644 |
--- a/components/password_manager/core/browser/password_form_manager.cc |
+++ b/components/password_manager/core/browser/password_form_manager.cc |
@@ -621,19 +621,14 @@ void PasswordFormManager::ProcessFrameInternal( |
// Proceed to autofill. |
// Note that we provide the choices but don't actually prefill a value if: |
- // (1) we are in Incognito mode, (2) the ACTION paths don't match, |
- // (3) if it matched using public suffix domain matching, or |
- // (4) the form is change password form. |
- // However, 2 and 3 should not apply to Android-based credentials found |
+ // (1) we are in Incognito mode, or |
+ // (2) if it matched using public suffix domain matching, or |
+ // (3) the form is change password form. |
+ // However, (2) should not apply to Android-based credentials found |
dvadym
2017/05/23 14:59:52
The comment mentions Android credentials, but the
vasilii
2017/05/24 12:48:36
Done.
|
// via affiliation-based matching (we want to autofill them). |
- // TODO(engedy): Clean this up. See: https://crbug.com/476519. |
- bool wait_for_username = |
- client_->IsIncognito() || |
- (!IsValidAndroidFacetURI(preferred_match_->signon_realm) && |
- (observed_form_.action.GetWithEmptyPath() != |
- preferred_match_->action.GetWithEmptyPath() || |
- preferred_match_->is_public_suffix_match || |
- observed_form_.IsPossibleChangePasswordForm())); |
+ bool wait_for_username = client_->IsIncognito() || |
+ preferred_match_->is_public_suffix_match || |
+ observed_form_.IsPossibleChangePasswordForm(); |
if (wait_for_username) { |
manager_action_ = kManagerActionNone; |
} else { |