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..0ed5aa0223b389074a505a2a9f40399cf3813889 100644 |
| --- a/components/autofill/core/browser/autofill_manager.cc |
| +++ b/components/autofill/core/browser/autofill_manager.cc |
| @@ -557,6 +557,9 @@ 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; |
|
Ilya Sherman
2014/06/03 20:39:31
nit: Please leave a blank line after this one.
|
| + // Mark the field as autofilled when a non-empty value is assigned to |
| + // it. |
|
Ilya Sherman
2014/06/03 20:39:31
nit: Please extend this comment with "This allows
|
| + result.fields[i].is_autofilled = true; |
| if (should_notify) |
| manager_delegate_->DidFillOrPreviewField(value, profile_full_name); |