Remove "Search" suffix from omnibox search engine match

Remove "Search" suffix from omnibox search engine match
Example (new on top, old on bottom): http://imgur.com/a/vx7pW

BUG=99426
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=203052

[Devlin, 2013-05-28 20:12:29]

lgtm

[Aaron Jacobs, 2013-05-29 20:22:04]

Peter: Owners for chrome/browser/autocomplete/autocomplete_controller.cc

[Peter Kasting, 2013-05-29 21:07:20]

LGTM

https://codereview.chromium.org/16019014/diff/2001/chrome/browser/autocomplet...
File chrome/browser/autocomplete/autocomplete_controller.cc (right):

https://codereview.chromium.org/16019014/diff/2001/chrome/browser/autocomplet...
chrome/browser/autocomplete/autocomplete_controller.cc:564: // Don't add the
"Search" suffix for extension keywords.
Nit: Shorter but with a more descriptive comment:

          // For extension keywords, just make the description the extension
          // name -- don't assume that the normal search keyword description is
          // applicable.
          i->description = template_url->AdjustedShortNameForLocaleDirection();
          if (!template_url->IsExtensionKeyword()) {
             i->description = l10n_util::GetStringFUTF16(
                 IDS_AUTOCOMPLETE_SEARCH_DESCRIPTION, i->description);
          }

[Aaron Jacobs, 2013-05-29 21:27:21]

https://codereview.chromium.org/16019014/diff/2001/chrome/browser/autocomplet...
File chrome/browser/autocomplete/autocomplete_controller.cc (right):

https://codereview.chromium.org/16019014/diff/2001/chrome/browser/autocomplet...
chrome/browser/autocomplete/autocomplete_controller.cc:564: // Don't add the
"Search" suffix for extension keywords.
On 2013/05/29 21:07:20, Peter Kasting wrote:
> Nit: Shorter but with a more descriptive comment:
> 
>           // For extension keywords, just make the description the extension
>           // name -- don't assume that the normal search keyword description
is
>           // applicable.
>           i->description =
template_url->AdjustedShortNameForLocaleDirection();
>           if (!template_url->IsExtensionKeyword()) {
>              i->description = l10n_util::GetStringFUTF16(
>                  IDS_AUTOCOMPLETE_SEARCH_DESCRIPTION, i->description);
>           }

Done.

[commit-bot: I haz the power, 2013-05-29 21:27:43]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/Samusaaron3@gmail.com/16019014/9001

[commit-bot: I haz the power, 2013-05-30 03:57:18]

Change committed as 203052