Polish the keybinding implementation a bit.

chrome/common/extensions/extension_unittest.cc

Index: chrome/common/extensions/extension_unittest.cc
===================================================================
--- chrome/common/extensions/extension_unittest.cc	(revision 127690)
+++ chrome/common/extensions/extension_unittest.cc	(working copy)
@@ -970,20 +970,18 @@
     // Case insensitivity is OK.
     { true, CtrlF, "command", "Ctrl+F", "description" },
     { true, CtrlF, "command", "cTrL+f", "description" },
-    // Extra spaces are fine.
-    { true, CtrlShiftF, "command", "  Ctrl + Shift   +F", "description" },
-    // Minus is equivalent to plus.
-    { true, CtrlShiftF, "command", "Ctrl+Shift-F", "description" },
     // Skipping description is OK for browser- and pageActions.
-    { true, CtrlF, "browserAction", "Ctrl+F", "" },
-    { true, CtrlF, "pageAction",    "Ctrl+F", "" },
+    { true, CtrlF, "_execute_browser_action", "Ctrl+F", "" },
+    { true, CtrlF, "_execute_page_action",   "Ctrl+F", "" },
   };
 
   // TODO(finnur): test Command/Options on Mac when implemented.
 
   for (size_t i = 0; i < ARRAYSIZE_UNSAFE(kTests); ++i) {
     scoped_ptr<DictionaryValue> command(new DictionaryValue);
-    command->SetString("key", kTests[i].key);
+    command->SetString("suggested_key_win", kTests[i].key);
+    command->SetString("suggested_key_mac", kTests[i].key);
+    command->SetString("suggested_key_other", kTests[i].key);
     command->SetString("description", kTests[i].description);
 
     Extension::ExtensionKeybinding keybinding;