Move final vector icons out of ui/gfx/vector_icons/ and remove the

components/omnibox/browser/autocomplete_match.cc

Index: components/omnibox/browser/autocomplete_match.cc
diff --git a/components/omnibox/browser/autocomplete_match.cc b/components/omnibox/browser/autocomplete_match.cc
index 9d9c7eaa1a014c074516fd8b9d97edc2fa52c78b..de13a63d3c4cab1cac2a683fd8e729af63ff4e61 100644
--- a/components/omnibox/browser/autocomplete_match.cc
+++ b/components/omnibox/browser/autocomplete_match.cc
@@ -26,6 +26,7 @@
 
 #if !defined(OS_ANDROID) && !defined(OS_IOS)
 #include "components/omnibox/browser/vector_icons.h"  // nogncheck
+#include "ui/vector_icons/vector_icons.h"             // nogncheck
 #endif
 
 namespace {
@@ -198,7 +199,7 @@ const gfx::VectorIcon& AutocompleteMatch::TypeToVectorIcon(Type type) {
     case Type::SEARCH_OTHER_ENGINE:
     case Type::CONTACT_DEPRECATED:
     case Type::VOICE_SUGGEST:
-      return omnibox::kSearchIcon;
+      return ui::kSearchIcon;

[sky, 2017/03/08 05:26:37]

I have a preference for keeping the icon close to the code. Is there a
particularly reason to move the icon to ui? What's wrong with keeping it here?

[Evan Stade, 2017/03/08 17:37:39]

this icon is used in:

ash/
chrome/browser/
components/omnibox/

previous to this CL it was duplicated so that ash could use it.

[sky, 2017/03/08 18:46:07]

Ok, makes sense.

 
     case Type::EXTENSION_APP:
       return omnibox::kExtensionAppIcon;