Add support for Command key for Extension Commands

ui/base/accelerators/accelerator.cc

Index: ui/base/accelerators/accelerator.cc
===================================================================
--- ui/base/accelerators/accelerator.cc	(revision 152319)
+++ ui/base/accelerators/accelerator.cc	(working copy)
@@ -82,6 +82,10 @@
   return (modifiers_ & EF_ALT_DOWN) == EF_ALT_DOWN;
 }
 
+bool Accelerator::IsCmdDown() const {
+  return (modifiers_ & EF_COMMAND_DOWN) == EF_COMMAND_DOWN;

[Nico, 2012/08/22 14:30:49]

nit: if you say `!= 0` instead of `== EF_COMMAND_DOWN`, you don't have to repeat
yourself, and you're less likely to make a copy-paste error later.

+}
+
 string16 Accelerator::GetShortcutText() const {
   int string_id = 0;
   switch(key_code_) {
@@ -194,6 +198,9 @@
   else if (IsAltDown())
     shortcut = l10n_util::GetStringFUTF16(IDS_APP_ALT_MODIFIER, shortcut);
 
+  if (IsCmdDown())
+    shortcut = l10n_util::GetStringFUTF16(IDS_APP_COMMAND_MODIFIER, shortcut);
+
   // For some reason, menus in Windows ignore standard Unicode directionality
   // marks (such as LRE, PDF, etc.). On RTL locales, we use RTL menus and
   // therefore any text we draw for the menu items is drawn in an RTL context.