Enforce "override bookmarks UI" manifest permissions for bookmark shortcut overrides

chrome/common/extensions/manifest_handlers/ui_overrides_handler.cc

Index: chrome/common/extensions/manifest_handlers/ui_overrides_handler.cc
diff --git a/chrome/common/extensions/manifest_handlers/ui_overrides_handler.cc b/chrome/common/extensions/manifest_handlers/ui_overrides_handler.cc
index 5e387da6909acf1b82ab1ded681f262dbae0ddec..1f8b6cd0a469dcb691d17bdb81084b1b2b454f11 100644
--- a/chrome/common/extensions/manifest_handlers/ui_overrides_handler.cc
+++ b/chrome/common/extensions/manifest_handlers/ui_overrides_handler.cc
@@ -184,7 +184,9 @@ bool UIOverridesHandler::Parse(Extension* extension, base::string16* error) {
   UIOverrides* assigned_ui_overrides = info.get();
   extension->SetManifestData(manifest_keys::kUIOverride, info.release());
   assigned_ui_overrides->manifest_permission.reset(new ManifestPermissionImpl(
-      UIOverrides::RemovesBookmarkButton(extension)));
+      UIOverrides::RemovesBookmarkButton(extension) ||
+      UIOverrides::RemovesBookmarkShortcut(extension) ||
+      UIOverrides::RemovesBookmarkOpenPagesShortcut(extension)));

[not at google - send to devlin, 2014/06/09 18:16:28]

can this basically be info->bookmarks_ui.get() (i.e. just "true" at this
point?).

[Mike Wittman, 2014/06/09 19:57:34]

Yes, I think so. Changed.

   return true;
 }