| Index: components/password_manager/core/browser/password_form_manager.h
|
| diff --git a/components/password_manager/core/browser/password_form_manager.h b/components/password_manager/core/browser/password_form_manager.h
|
| index 9256b35331ccafc6218a82243cb4bae4db296db3..91f53a454a84ff2daea247f5610f7b24201c6c3e 100644
|
| --- a/components/password_manager/core/browser/password_form_manager.h
|
| +++ b/components/password_manager/core/browser/password_form_manager.h
|
| @@ -86,13 +86,16 @@ class PasswordFormManager : public PasswordStoreConsumer {
|
|
|
| // Returns true if the observed form has both the current and new password
|
| // fields, and the username field was not explicitly marked with
|
| - // autocomplete=username. In these cases it is not clear whether the username
|
| - // field is the right guess (often such change password forms do not contain
|
| - // the username at all), and the user should not be bothered with saving a
|
| - // potentially malformed credential. Once we handle change password forms
|
| - // correctly, or http://crbug.com/448351 gets implemented, this method should
|
| - // be replaced accordingly.
|
| - bool IsIgnorableChangePasswordForm() const;
|
| + // "autocomplete=username" and the user-typed username and current password
|
| + // field values do not match the credentials already stored. In these cases it
|
| + // is not clear whether the username field is the right guess (often such
|
| + // change password forms do not contain the username at all), and the user
|
| + // should not be bothered with saving a potentially malformed credential. Once
|
| + // we handle change password forms correctly, this method should be replaced
|
| + // accordingly.
|
| + bool IsIgnorableChangePasswordForm(
|
| + const base::string16& typed_username,
|
| + const base::string16& typed_password) const;
|
|
|
| // Determines if the user opted to 'never remember' passwords for this form.
|
| bool IsBlacklisted() const;
|
|
|