Chromium Code Reviews| Index: components/autofill/core/common/password_form.h |
| diff --git a/components/autofill/core/common/password_form.h b/components/autofill/core/common/password_form.h |
| index f58222d94cc0f1dcd7d9ba9a856b62dc7ab3711b..50de50934c9cfb30d23a36c892b59e25a3cef5f9 100644 |
| --- a/components/autofill/core/common/password_form.h |
| +++ b/components/autofill/core/common/password_form.h |
| @@ -84,24 +84,6 @@ struct PasswordForm { |
| // data from the database, so it must not be empty. |
| std::string signon_realm; |
| - // The original "Realm" for the sign-on (scheme, host, port for SCHEME_HTML, |
| - // and contains the HTTP realm for dialog-based forms). This realm is only set |
| - // when two PasswordForms are matched when trying to find a login/pass pair |
| - // for a site. It is only set to a non-empty value during a match of the |
| - // original stored login/pass and the current observed form if all these |
| - // statements are true: |
| - // 1) The full signon_realm is not the same. |
| - // 2) The registry controlled domain is the same. For example; example.com, |
| - // m.example.com, foo.login.example.com and www.example.com would all resolve |
| - // to example.com since .com is the public suffix. |
| - // 3) The scheme is the same. |
| - // 4) The port is the same. |
| - // For example, if there exists a stored password for http://www.example.com |
| - // (where .com is the public suffix) and the observed form is |
| - // http://m.example.com, |original_signon_realm| must be set to |
| - // http://www.example.com. |
| - std::string original_signon_realm; |
| - |
| // An origin URL consists of the scheme, host, port and path; the rest is |
| // stripped. This is the primary data used by the PasswordManager to decide |
| // (in longest matching prefix fashion) whether or not a given PasswordForm |
| @@ -279,8 +261,8 @@ struct PasswordForm { |
| // TODO(vabr): Remove |is_alive| once http://crbug.com/486931 is fixed. |
| bool is_alive; // Set on construction, reset on destruction. |
| - // Returns true if this match was found using public suffix matching. |
| - bool IsPublicSuffixMatch() const; |
| + // true if this match was found using public suffix matching. |
|
engedy
2015/09/29 16:20:45
comment nit: If true, this match ...
dvadym
2015/09/29 18:29:36
Done.
|
| + bool is_public_suffix_match; |
| // Return true if we consider this form to be a change password form. |
| // We use only client heuristics, so it could include signup forms. |