Chromium Code Reviews| Index: components/omnibox/browser/autocomplete_match.cc |
| diff --git a/components/omnibox/browser/autocomplete_match.cc b/components/omnibox/browser/autocomplete_match.cc |
| index de13a63d3c4cab1cac2a683fd8e729af63ff4e61..4e420a621eb87e9768fba1ec6b89972e097c5652 100644 |
| --- a/components/omnibox/browser/autocomplete_match.cc |
| +++ b/components/omnibox/browser/autocomplete_match.cc |
| @@ -123,16 +123,18 @@ AutocompleteMatch::AutocompleteMatch(const AutocompleteMatch& match) |
| answer(SuggestionAnswer::copy(match.answer.get())), |
| transition(match.transition), |
| type(match.type), |
| - associated_keyword(match.associated_keyword.get() ? |
| - new AutocompleteMatch(*match.associated_keyword) : NULL), |
| + specific_type_identifier(match.specific_type_identifier), |
| + associated_keyword(match.associated_keyword.get() |
| + ? new AutocompleteMatch(*match.associated_keyword) |
| + : NULL), |
| keyword(match.keyword), |
| from_previous(match.from_previous), |
| - search_terms_args(match.search_terms_args.get() ? |
| - new TemplateURLRef::SearchTermsArgs(*match.search_terms_args) : |
| - NULL), |
| + search_terms_args( |
| + match.search_terms_args.get() |
| + ? new TemplateURLRef::SearchTermsArgs(*match.search_terms_args) |
| + : NULL), |
| additional_info(match.additional_info), |
| - duplicate_matches(match.duplicate_matches) { |
| -} |
| + duplicate_matches(match.duplicate_matches) {} |
| AutocompleteMatch::~AutocompleteMatch() { |
| } |
| @@ -161,6 +163,7 @@ AutocompleteMatch& AutocompleteMatch::operator=( |
| answer = SuggestionAnswer::copy(match.answer.get()); |
| transition = match.transition; |
| type = match.type; |
| + specific_type_identifier = match.specific_type_identifier; |
|
Mark P
2017/03/21 19:54:24
Nice work. Most times I review changes to Autocom
|
| associated_keyword.reset(match.associated_keyword.get() ? |
| new AutocompleteMatch(*match.associated_keyword) : NULL); |
| keyword = match.keyword; |