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

Side by Side Diff: components/autofill/core/common/password_form.h

Issue 2637823002: [Password Generation] Send votes about confirmation fields (Closed)
Patch Set: Changes addressed to reviewer comments Created 3 years, 11 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 unified diff | Download patch
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef COMPONENTS_AUTOFILL_CORE_COMMON_PASSWORD_FORM_H__ 5 #ifndef COMPONENTS_AUTOFILL_CORE_COMMON_PASSWORD_FORM_H__
6 #define COMPONENTS_AUTOFILL_CORE_COMMON_PASSWORD_FORM_H__ 6 #define COMPONENTS_AUTOFILL_CORE_COMMON_PASSWORD_FORM_H__
7 7
8 #include <map> 8 #include <map>
9 #include <memory> 9 #include <memory>
10 #include <string> 10 #include <string>
(...skipping 147 matching lines...) Expand 10 before | Expand all | Expand 10 after
158 base::string16 password_value; 158 base::string16 password_value;
159 159
160 // Whether the password value is the same as specified in the "value" 160 // Whether the password value is the same as specified in the "value"
161 // attribute of the input element. Only used in the renderer. 161 // attribute of the input element. Only used in the renderer.
162 bool password_value_is_default; 162 bool password_value_is_default;
163 163
164 // If the form was a sign-up or a change password form, the name of the input 164 // If the form was a sign-up or a change password form, the name of the input
165 // element corresponding to the new password. Optional, and not persisted. 165 // element corresponding to the new password. Optional, and not persisted.
166 base::string16 new_password_element; 166 base::string16 new_password_element;
167 167
168 // If |new_password_element| is defined and there is another password element
169 // with the same user input, keep the name of the element in
dvadym 2017/01/17 16:32:21 Nit: I'd propose to simplify this comment, somethi
kolos1 2017/01/18 09:15:14 Done.
170 // |password_confirmation_element|.
171 base::string16 confirmation_password_element;
172
168 // The new password. Optional, and not persisted. 173 // The new password. Optional, and not persisted.
169 base::string16 new_password_value; 174 base::string16 new_password_value;
170 175
171 // Whether the password value is the same as specified in the "value" 176 // Whether the password value is the same as specified in the "value"
172 // attribute of the input element. Only used in the renderer. 177 // attribute of the input element. Only used in the renderer.
173 bool new_password_value_is_default; 178 bool new_password_value_is_default;
174 179
175 // Whether the |new_password_element| has an autocomplete=new-password 180 // Whether the |new_password_element| has an autocomplete=new-password
176 // attribute. This is only used in parsed HTML forms. 181 // attribute. This is only used in parsed HTML forms.
177 bool new_password_marked_by_site; 182 bool new_password_marked_by_site;
(...skipping 120 matching lines...) Expand 10 before | Expand all | Expand 10 after
298 }; 303 };
299 304
300 // For testing. 305 // For testing.
301 std::ostream& operator<<(std::ostream& os, PasswordForm::Layout layout); 306 std::ostream& operator<<(std::ostream& os, PasswordForm::Layout layout);
302 std::ostream& operator<<(std::ostream& os, const PasswordForm& form); 307 std::ostream& operator<<(std::ostream& os, const PasswordForm& form);
303 std::ostream& operator<<(std::ostream& os, PasswordForm* form); 308 std::ostream& operator<<(std::ostream& os, PasswordForm* form);
304 309
305 } // namespace autofill 310 } // namespace autofill
306 311
307 #endif // COMPONENTS_AUTOFILL_CORE_COMMON_PASSWORD_FORM_H__ 312 #endif // COMPONENTS_AUTOFILL_CORE_COMMON_PASSWORD_FORM_H__
OLDNEW
« no previous file with comments | « components/autofill/core/browser/field_types.h ('k') | components/password_manager/core/browser/password_form_manager.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698