DevTools: Decouple shortcuts from actions, introduce shortcut contexts

Source/devtools/front_end/modules.js

 /*
  * Copyright (C) 2014 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  * 1. Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *
@@ skipping 17 matching lines @@
 
 /**
  * @type {!Array.<!WebInspector.ModuleManager.ModuleDescriptor>}
  */
 var allDescriptors = [
     {
         name: "main",
         extensions: [
             {
                 type: "@WebInspector.ActionDelegate",
+                actionId: "main.reload",
+                className: "WebInspector.Main.ReloadActionDelegate",
                 bindings: [
                     {
                         platform: "windows,linux",
                         shortcut: "F5 Ctrl+R"
                     },
                     {
                         platform: "mac",
                         shortcut: "Meta+R"
                     }
-                ],
-                className: "WebInspector.Main.ReloadActionDelegate"
+                ]
             },
             {
                 type: "@WebInspector.ActionDelegate",
+                actionId: "main.hard-reload",
+                className: "WebInspector.Main.HardReloadActionDelegate",
                 bindings: [
                     {
                         platform: "windows,linux",
                         shortcut: "Shift+F5 Ctrl+F5 Ctrl+Shift+F5 Shift+Ctrl+R"
                     },
                     {
                         platform: "mac",
                         shortcut: "Shift+Meta+R"
                     }
-                ],
-                className: "WebInspector.Main.HardReloadActionDelegate"
+                ]
             },
             {
                 type: "@WebInspector.ActionDelegate",
+                actionId: "main.toggle-drawer",
+                className: "WebInspector.InspectorView.DrawerToggleActionDelegate",
                 bindings: [
                     {
                         shortcut: "Esc"
                     }
-                ],
-                className: "WebInspector.InspectorView.DrawerToggleActionDelegate"
+                ]
             },
             {
                 type: "@WebInspector.ActionDelegate",
+                actionId: "main.debug-reload",
+                className: "WebInspector.Main.DebugReloadActionDelegate",
                 bindings: [
                     {
-                        shortcut: "Alt+R"
+                        shortcut: "Alt+R",
                     }
-                ],
-                className: "WebInspector.Main.DebugReloadActionDelegate"
+                ]
             },
             {
                 type: "ui-setting",
                 title: "Disable cache (while DevTools is open)",
                 settingName: "cacheDisabled",
                 settingType: "checkbox"
             },
             {
                 type: "ui-setting",
                 section: "Appearance",
@@ skipping 205 matching lines @@
             },
             {
                 type: "navigator-view",
                 name: "snippets",
                 title: "Snippets",
                 order: 3,
                 className: "WebInspector.SnippetsNavigatorView"
             },
             {
                 type: "@WebInspector.ActionDelegate",
+                actionId: "sources.go-to-source",
+                className: "WebInspector.SourcesPanel.ShowGoToSourceDialogActionDelegate",
                 bindings: [
                     {
                         platform: "mac",
                         shortcut: "Meta+O Meta+P"
                     },
                     {
                         platform: "windows,linux",
                         shortcut: "Ctrl+O Ctrl+P"
                     }
-                ],
-                className: "WebInspector.SourcesPanel.ShowGoToSourceDialogActionDelegate"
+                ]
             },
             {
                 type: "ui-setting",
                 settingName: "javaScriptDisabled",
                 settingType: "custom",
                 className: "WebInspector.SourcesPanel.DisableJavaScriptSettingDelegate"
             },
             {
                 type: "ui-setting",
                 section: "Sources",
@@ skipping 161 matching lines @@
                 order: "0",
                 className: "WebInspector.ConsolePanel.WrapperView"
             },
             {
                 type: "@WebInspector.Revealer",
                 contextTypes: ["WebInspector.ConsoleModel"],
                 className: "WebInspector.ConsolePanel.ConsoleRevealer"
             },
             {
                 type: "@WebInspector.ActionDelegate",
+                actionId: "console.show",
+                className: "WebInspector.ConsoleView.ShowConsoleActionDelegate",
                 bindings: [
                     {
                         shortcut: "Ctrl+`"
                     }
-                ],
-                className: "WebInspector.ConsoleView.ShowConsoleActionDelegate"
+                ]
             },
             {
                 type: "ui-setting",
                 section: "Console",
                 title: "Log XMLHttpRequests",
                 settingName: "monitoringXHREnabled",
                 settingType: "checkbox"
             },
             {
                 type: "ui-setting",
@@ skipping 17 matching lines @@
         extensions: [
              {
                  type: "drawer-view",
                  name: "search",
                  title: "Search",
                  order: "1",
                  className: "WebInspector.AdvancedSearchView"
              },
              {
                  type: "@WebInspector.ActionDelegate",
+                 actionId: "search.toggle",
+                 className: "WebInspector.AdvancedSearchView.ToggleDrawerViewActionDelegate",
                  bindings: [
                      {
                          platform: "mac",
                          shortcut: "Meta+Alt+F"
                      },
                      {
                          platform: "windows,linux",
                          shortcut: "Ctrl+Shift+F"
                      }
-                 ],
-                 className: "WebInspector.AdvancedSearchView.ToggleDrawerViewActionDelegate"
+                 ]
              }
         ],
         scripts: [ "AdvancedSearchView.js" ]
     },
     {
         name: "settings",
         extensions: [
             {
                 type: "@WebInspector.ActionDelegate",
+                actionId: "settings.show",
+                className: "WebInspector.SettingsController.SettingsScreenActionDelegate",
                 bindings: [
                     {
                         shortcut: "F1 Shift+?"
                     }
-                ],
-                className: "WebInspector.SettingsController.SettingsScreenActionDelegate"
+                ]
             }
         ]
     },
     {
         name: "extensions",
         extensions: [
             {
                 type: "@WebInspector.ExtensionServerAPI",
                 className: "WebInspector.ExtensionServer"
             }
@@ skipping 22 matching lines @@
         name: "handler-registry",
         extensions: [
             {
                 type: "@WebInspector.ContextMenu.Provider",
                 contextTypes: ["WebInspector.UISourceCode", "WebInspector.Resource", "WebInspector.NetworkRequest", "Node"],
                 className: "WebInspector.HandlerRegistry.ContextMenuProvider"
             }
         ]
     }
 ];