Chromium Code Reviews| Index: components/autofill/core/browser/autofill_external_delegate.cc |
| diff --git a/components/autofill/core/browser/autofill_external_delegate.cc b/components/autofill/core/browser/autofill_external_delegate.cc |
| index c4367567ff274c7801e20dea6ef5f0dd89111e21..6dde33d468d9b13016a95e0bec3fb867ecadd6b0 100644 |
| --- a/components/autofill/core/browser/autofill_external_delegate.cc |
| +++ b/components/autofill/core/browser/autofill_external_delegate.cc |
| @@ -164,12 +164,18 @@ bool AutofillExternalDelegate::ShouldRepostEvent(const ui::MouseEvent& event) { |
| return true; |
| } |
| -void AutofillExternalDelegate::DidSelectSuggestion(int identifier) { |
| +void AutofillExternalDelegate::DidSelectSuggestion(int identifier, |
| + const base::string16& value) |
| +{ |
|
Ilya Sherman
2014/02/22 05:59:51
nit: The curly brace should be on the previous lin
ziran.sun
2014/02/27 15:38:11
Done.
|
| ClearPreviewedForm(); |
| // Only preview the data if it is a profile. |
| if (identifier > 0) |
| FillAutofillFormData(identifier, true); |
| + // Preview Autocomplete field |
|
Ilya Sherman
2014/02/22 05:59:51
nit: This comment is redundant with the code; plea
|
| + else if (identifier == WebAutofillClient::MenuItemIDAutocompleteEntry) { |
| + autofill_driver_->RendererShouldPreviewAutoCompleteNode(value); |
| + } |
|
Ilya Sherman
2014/02/22 05:59:51
nit: No need for curly braces.
ziran.sun
2014/02/27 15:38:11
Done.
|
| } |
| void AutofillExternalDelegate::DidAcceptSuggestion(const base::string16& value, |