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

Unified Diff: chrome/browser/autofill/form_structure.cc

Issue 10669038: base: Remove dereference structure operator (i.e ->) from ScopedVector. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 8 years, 6 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: chrome/browser/autofill/form_structure.cc
diff --git a/chrome/browser/autofill/form_structure.cc b/chrome/browser/autofill/form_structure.cc
index 1c8b7c95d786583b312422ccb242cc6f36501473..06183a6a4b7a6dc1193811dfd0d4e0860c6a8bf2 100644
--- a/chrome/browser/autofill/form_structure.cc
+++ b/chrome/browser/autofill/form_structure.cc
@@ -886,8 +886,8 @@ void FormStructure::ParseAutocompletetypeAttributes(bool* found_attribute,
bool* found_sections) {
*found_attribute = false;
*found_sections = false;
- for (std::vector<AutofillField*>::iterator field = fields_->begin();
- field != fields_->end(); ++field) {
+ for (std::vector<AutofillField*>::iterator field = fields_.begin();
+ field != fields_.end(); ++field) {
if ((*field)->autocomplete_type.empty())
continue;
@@ -917,14 +917,14 @@ void FormStructure::IdentifySections(bool has_author_specified_sections) {
if (!has_author_specified_sections) {
// Name sections after the first field in the section.
- string16 current_section = fields_->front()->unique_name();
+ string16 current_section = fields_.front()->unique_name();
// Keep track of the types we've seen in this section.
std::set<AutofillFieldType> seen_types;
AutofillFieldType previous_type = UNKNOWN_TYPE;
- for (std::vector<AutofillField*>::iterator field = fields_->begin();
- field != fields_->end(); ++field) {
+ for (std::vector<AutofillField*>::iterator field = fields_.begin();
+ field != fields_.end(); ++field) {
const AutofillFieldType current_type =
AutofillType::GetEquivalentFieldType((*field)->type());
@@ -963,8 +963,8 @@ void FormStructure::IdentifySections(bool has_author_specified_sections) {
// Ensure that credit card and address fields are in separate sections.
// This simplifies the section-aware logic in autofill_manager.cc.
- for (std::vector<AutofillField*>::iterator field = fields_->begin();
- field != fields_->end(); ++field) {
+ for (std::vector<AutofillField*>::iterator field = fields_.begin();
+ field != fields_.end(); ++field) {
AutofillType::FieldTypeGroup field_type_group =
AutofillType((*field)->type()).group();
if (field_type_group == AutofillType::CREDIT_CARD)
« no previous file with comments | « chrome/browser/autofill/autofill_manager_unittest.cc ('k') | chrome/browser/autofill/personal_data_manager.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698