Update omnibox chips in MD

chrome/browser/ui/views/location_bar/selected_keyword_view.cc

Index: chrome/browser/ui/views/location_bar/selected_keyword_view.cc
diff --git a/chrome/browser/ui/views/location_bar/selected_keyword_view.cc b/chrome/browser/ui/views/location_bar/selected_keyword_view.cc
index fc03c1f925630c79519f2cac109c7cce1f15e568..b25f5090c379778997d7c3f154287d761f2a1ab6 100644
--- a/chrome/browser/ui/views/location_bar/selected_keyword_view.cc
+++ b/chrome/browser/ui/views/location_bar/selected_keyword_view.cc
@@ -56,10 +56,8 @@ SkColor SelectedKeywordView::GetTextColor() const {
   if (!ui::MaterialDesignController::IsModeMaterial())
     return text_color_;
 
-  return color_utils::IsDark(GetParentBackgroundColor())
-             ? gfx::kGoogleBlue700
-             : GetNativeTheme()->GetSystemColor(
-                   ui::NativeTheme::kColorId_LinkEnabled);
+  return GetNativeTheme()->GetSystemColor(
+      ui::NativeTheme::kColorId_LinkEnabled);
 }
 
 SkColor SelectedKeywordView::GetBorderColor() const {
@@ -96,16 +94,19 @@ void SelectedKeywordView::SetKeyword(const base::string16& keyword) {
   bool is_extension_keyword;
   const base::string16 short_name =
       model->GetKeywordShortName(keyword, &is_extension_keyword);
-  const base::string16 full_name = is_extension_keyword ?
-      short_name :
-      l10n_util::GetStringFUTF16(IDS_OMNIBOX_KEYWORD_TEXT, short_name);
+  int keyword_text_ids = ui::MaterialDesignController::IsModeMaterial()

[Peter Kasting, 2016/05/20 20:54:11]

Nit: "ids" here looks to me more like "IDs" than "IDS" -- I might just call this
"id".  That matches other code where we e.g. use "id" instead of "idr" in
graphic resource variable names.

[Evan Stade, 2016/05/20 22:33:53]

Done.

+                             ? IDS_OMNIBOX_KEYWORD_TEXT_MD
+                             : IDS_OMNIBOX_KEYWORD_TEXT;
+  const base::string16 full_name =
+      is_extension_keyword ? short_name : l10n_util::GetStringFUTF16(
+                                              keyword_text_ids, short_name);
   full_label_.SetText(full_name);
 
   const base::string16 min_string(
       location_bar_util::CalculateMinString(short_name));
-  const base::string16 partial_name = is_extension_keyword ?
-      min_string :
-      l10n_util::GetStringFUTF16(IDS_OMNIBOX_KEYWORD_TEXT, min_string);
+  const base::string16 partial_name =
+      is_extension_keyword ? min_string : l10n_util::GetStringFUTF16(
+                                              keyword_text_ids, min_string);
   partial_label_.SetText(min_string.empty() ?
       full_label_.text() : partial_name);
 }