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

Unified Diff: components/autofill/browser/form_structure.h

Issue 12721004: Autofill:Autocomplete: Enable autocheckout of input elements of type password. This will support fi… (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Updated expectations for hueristic tests Created 7 years, 9 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/browser/form_structure.h
diff --git a/components/autofill/browser/form_structure.h b/components/autofill/browser/form_structure.h
index 731fe5c1db3a1a9a6f9648c3ac8110bf0288c544..90955148a8ccc3ffce20d6852620c5a3f4e0f192 100644
--- a/components/autofill/browser/form_structure.h
+++ b/components/autofill/browser/form_structure.h
@@ -136,7 +136,6 @@ class FormStructure {
const AutofillField* field(size_t index) const;
AutofillField* field(size_t index);
size_t field_count() const;
- size_t checkable_field_count() const;
// Returns the number of fields that are able to be autofilled.
size_t autofill_count() const { return autofill_count_; }
@@ -192,6 +191,7 @@ class FormStructure {
// Returns the minimal number of fillable fields required to start autofill.
size_t RequiredFillableFields() const;
+ size_t active_field_count() const;
// The name of the form.
string16 form_name_;
@@ -208,8 +208,9 @@ class FormStructure {
// A vector of all the input fields in the form.
ScopedVector<AutofillField> fields_;
- // The number of fields able to be checked.
- size_t checkable_field_count_;
+ // The number of fields counted towards form signature and request to Autofill
+ // server.
+ size_t active_field_count_;
// The names of the form input elements, that are part of the form signature.
// The string starts with "&" and the names are also separated by the "&"

Powered by Google App Engine
This is Rietveld 408576698