Switch controlled settings indicators to inlined SVGs.

chrome/browser/extensions/extension_context_menu_model.cc

Index: chrome/browser/extensions/extension_context_menu_model.cc
diff --git a/chrome/browser/extensions/extension_context_menu_model.cc b/chrome/browser/extensions/extension_context_menu_model.cc
index ad638b7c21dee847d4554c2b2c53e8b2714a8d5f..2cfd4950ef6b24a1703fc814e40ed58c51ca70fb 100644
--- a/chrome/browser/extensions/extension_context_menu_model.cc
+++ b/chrome/browser/extensions/extension_context_menu_model.cc
@@ -312,7 +312,7 @@ void ExtensionContextMenuModel::InitMenu(const Extension* extension,
       int uninstall_index = GetIndexOfCommandId(UNINSTALL);
       SetIcon(uninstall_index,
               ui::ResourceBundle::GetSharedInstance().GetImageNamed(
-                  IDR_CONTROLLED_SETTING_MANDATORY));
+                  IDR_OMNIBOX_HTTPS_POLICY_WARNING));

[Bernhard Bauer, 2015/07/10 10:46:37]

This seems a bit strange, and somewhat brittle, e.g. if the omnibox icon is
changed in the future.

If this is just about using the same warning icon, should we rename the icon
name to reflect that?

[Evan Stade, 2015/07/10 19:04:16]

We probably want this to change if the omnibox icon changes (to maintain
consistency). My thinking was that this was no more brittle than it used to be
because if the icons were changed in chrome://settings this one would also
magically change, for better or worse. If you prefer I guess I can leave the old
identifier and use it here, as it doesn't change the size of the pak file either
way.

[Bernhard Bauer, 2015/07/10 20:27:31]

Oh, I didn't realize this is actually a policy-related icon for the omnibox, not
a generic warning. In which case, either is fine with me.

     }
   }