Add "CommandOrControl" to represent "Command" on OS X and "Ctrl" on other platforms.
authorCheng Zhao <zcbenz@gmail.com>
Fri, 29 Nov 2013 08:10:32 +0000 (16:10 +0800)
committerCheng Zhao <zcbenz@gmail.com>
Fri, 29 Nov 2013 08:11:48 +0000 (16:11 +0800)
browser/ui/accelerator_util.cc
docs/api/browser/menu-item.md

index 1fb89a7..b2aadb9 100644 (file)
@@ -109,6 +109,12 @@ bool StringToAccelerator(const std::string& description,
       modifiers |= ui::EF_CONTROL_DOWN;
     } else if (tokens[i] == "command") {
       modifiers |= ui::EF_COMMAND_DOWN;
+    } else if (tokens[i] == "commandorcontrol" || tokens[i] == "cmdorctrl") {
+#if defined(OS_MACOSX)
+      modifiers |= ui::EF_COMMAND_DOWN;
+#else
+      modifiers |= ui::EF_CONTROL_DOWN;
+#endif
     } else if (tokens[i] == "alt") {
       modifiers |= ui::EF_ALT_DOWN;
     } else if (tokens[i] == "shift") {
index 6ad24d9..ddf8d13 100644 (file)
@@ -23,4 +23,7 @@
 
 ## Notes on accelerator
 
-On Linux and Windows, the `Command` key would not have any effect.
+On Linux and Windows, the `Command` key would not have any effect, you can
+use `CommandOrControl` which represents `Command` on OS X and `Control` on
+Linux and Windows to define some accelerators, you can also use its short
+alias `CmdOrCtrl`.