Chromium Code Reviews| Index: chrome/browser/ui/autofill/autofill_dialog_controller_unittest.cc |
| diff --git a/chrome/browser/ui/autofill/autofill_dialog_controller_unittest.cc b/chrome/browser/ui/autofill/autofill_dialog_controller_unittest.cc |
| index b14193aee695d082d2e9aae680d71dde508a8fea..f0797350b524baa3a8df13dbcd122f627a017993 100644 |
| --- a/chrome/browser/ui/autofill/autofill_dialog_controller_unittest.cc |
| +++ b/chrome/browser/ui/autofill/autofill_dialog_controller_unittest.cc |
| @@ -743,8 +743,8 @@ TEST_F(AutofillDialogControllerTest, PhoneNumberValidation) { |
| const DetailInputs& inputs = |
| controller()->RequestedFieldsForSection(section); |
| AutofillProfile full_profile(test::GetVerifiedProfile()); |
| - for (size_t i = 0; i < inputs.size(); ++i) { |
| - const ServerFieldType type = inputs[i].type; |
| + for (size_t ix = 0; ix < inputs.size(); ++ix) { |
|
Ilya Sherman
2015/01/28 01:35:48
nit: Please use 'j' instead of 'ix'
|
| + const ServerFieldType type = inputs[ix].type; |
| outputs[type] = full_profile.GetInfo(AutofillType(type), "en-US"); |
| } |
| @@ -2966,10 +2966,10 @@ TEST_F(AutofillDialogControllerTest, InputEditability) { |
| controller()->OnDidGetWalletItems(items.Pass()); |
| EXPECT_TRUE(controller()->IsEditingExistingData(SECTION_CC_BILLING)); |
| - const DetailInputs& inputs = |
| - controller()->RequestedFieldsForSection(SECTION_CC_BILLING); |
| FieldValueMap outputs; |
| - CopyInitialValues(inputs, &outputs); |
| + CopyInitialValues( |
| + controller()->RequestedFieldsForSection(SECTION_CC_BILLING), |
| + &outputs); |
| controller()->GetView()->SetUserInput(SECTION_CC_BILLING, outputs); |
| for (size_t i = 0; i < arraysize(sections); ++i) { |