Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(235)

Unified Diff: components/autofill/core/common/password_form.h

Issue 1314903003: Updating of all entries in PasswordManager of the same credentials on password update (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Addressed Vasilii comments Created 5 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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.

Powered by Google App Engine
This is Rietveld 408576698