Chromium Code Reviews| Index: chrome/browser/autofill/autofill_scanner.h |
| diff --git a/chrome/browser/autofill/autofill_scanner.h b/chrome/browser/autofill/autofill_scanner.h |
| index 186e864deaa4a6f1433d216629c4a8f5f70ecb83..4b4644706f83266c9dcca2260c6ee0108db4507c 100644 |
| --- a/chrome/browser/autofill/autofill_scanner.h |
| +++ b/chrome/browser/autofill/autofill_scanner.h |
| @@ -13,6 +13,9 @@ |
| class AutofillField; |
| // A helper class for parsing a stream of |AutofillField|'s with lookahead. |
| +// Ignores checkable AutofillFields as they interfere when parsers assume |
| +// correct context (Eg., while parsing address, "Is PO box" checkbox |
| +// interferes with correctly understanding ADDRESS_LINE2). |
|
Ilya Sherman
2012/12/15 01:08:36
Rather than changing the scanner class, please cha
Raman Kakilate
2012/12/17 20:11:01
Done.
|
| class AutofillScanner { |
| public: |
| explicit AutofillScanner(const std::vector<const AutofillField*>& fields); |
| @@ -23,10 +26,10 @@ class AutofillScanner { |
| // Returns the current field in the stream, or |NULL| if there are no more |
| // fields in the stream. |
| - const AutofillField* Cursor() const; |
| + const AutofillField* Cursor(); |
| // Returns |true| if the cursor has reached the end of the stream. |
| - bool IsEnd() const; |
| + bool IsEnd(); |
| // Restores the most recently saved cursor. See also |SaveCursor()|. |
| void Rewind(); |
| @@ -51,6 +54,9 @@ class AutofillScanner { |
| // The past-the-end pointer for the stream. |
| const std::vector<const AutofillField*>::const_iterator end_; |
| + // Skip all clickable fields. |
| + void SkipClickables(); |
| + |
| DISALLOW_COPY_AND_ASSIGN(AutofillScanner); |
| }; |