Chromium Code Reviews| Index: components/autofill/core/browser/autofill_manager.cc |
| diff --git a/components/autofill/core/browser/autofill_manager.cc b/components/autofill/core/browser/autofill_manager.cc |
| index bfb8d492ca0e0cdfd936c8c9095c1ad72184f021..bc5356b97df92b75f98fa8e8ecb7247c97c0f3ac 100644 |
| --- a/components/autofill/core/browser/autofill_manager.cc |
| +++ b/components/autofill/core/browser/autofill_manager.cc |
| @@ -557,6 +557,10 @@ void AutofillManager::FillOrPreviewForm( |
| // Mark the cached field as autofilled, so that we can detect when a |
| // user edits an autofilled field (for metrics). |
| form_structure->field(i)->is_autofilled = true; |
| + // Mark the "select-one" field as autofilled when a non-empty value is |
| + // assigned to it. |
|
Ilya Sherman
2014/06/02 23:51:00
Is there any harm to doing this for all the autofi
|
| + if (result.fields[i].form_control_type == "select-one") |
| + result.fields[i].is_autofilled = true; |
| if (should_notify) |
| manager_delegate_->DidFillOrPreviewField(value, profile_full_name); |