Chromium Code Reviews| Index: chrome/browser/autocomplete/autocomplete_match.cc |
| diff --git a/chrome/browser/autocomplete/autocomplete_match.cc b/chrome/browser/autocomplete/autocomplete_match.cc |
| index e5e2cd3ee8fb13a4ed26870ad665471188c60ebe..0fd0db06474d493733e925d096bb6ebe355aaf14 100644 |
| --- a/chrome/browser/autocomplete/autocomplete_match.cc |
| +++ b/chrome/browser/autocomplete/autocomplete_match.cc |
| @@ -73,6 +73,9 @@ AutocompleteMatch::AutocompleteMatch(const AutocompleteMatch& match) |
| from_previous(match.from_previous) { |
| if (match.associated_keyword.get()) |
|
Peter Kasting
2012/06/22 21:48:31
Nit: For consistency, we could probably change the
Bart N
2012/06/22 22:42:01
Actually, to satisfy both performance & consistenc
|
| associated_keyword.reset(new AutocompleteMatch(*match.associated_keyword)); |
| + if (match.search_terms_args.get()) |
| + search_terms_args.reset( |
| + new TemplateURLRef::SearchTermsArgs(*match.search_terms_args)); |
| } |
| AutocompleteMatch::~AutocompleteMatch() { |
| @@ -103,7 +106,8 @@ AutocompleteMatch& AutocompleteMatch::operator=( |
| keyword = match.keyword; |
| starred = match.starred; |
| from_previous = match.from_previous; |
| - |
| + search_terms_args.reset(match.search_terms_args.get() ? |
| + new TemplateURLRef::SearchTermsArgs(*match.search_terms_args) : NULL); |
| return *this; |
| } |