DevTools: do not consider ? a shortcut while editing.

Source/devtools/front_end/ui/ShortcutRegistry.js

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 /**
  * @constructor
  * @param {!WebInspector.ActionRegistry} actionRegistry
  * @param {!Document} document
  */
 WebInspector.ShortcutRegistry = function(actionRegistry, document)
@@ skipping 74 matching lines @@
 
     /**
      * @param {number} key
      * @param {string} keyIdentifier
      * @param {!KeyboardEvent=} event
      */
     handleKey: function(key, keyIdentifier, event)
     {
         var keyModifiers = key >> 8;
         var actionIds = this.applicableActions(key);
+        if (!actionIds.length)
+            return;
         if (WebInspector.GlassPane.DefaultFocusedViewStack.length > 1) {
-            if (event && actionIds.length && !isPossiblyInputKey())
+            if (event && !isPossiblyInputKey())
                 event.consume(true);
             return;
         }
 
-        if (!isPossiblyInputKey())
+        if (!isPossiblyInputKey()) {
+            if (event)
+                event.consume(true);
             processActionIdsSequentially.call(this);
-        else
+        } else {
             this._pendingActionTimer = setTimeout(processActionIdsSequentially.bind(this), 0);
+        }
 
         /**
          * @this {WebInspector.ShortcutRegistry}
          */
         function processActionIdsSequentially()
         {
             delete this._pendingActionTimer;
             var actionId = actionIds.shift();
             if (!actionId)
                 return;
 
             this._actionRegistry.execute(actionId).then(continueIfNecessary.bind(this));
 
             /**
              * @this {WebInspector.ShortcutRegistry}
              */
             function continueIfNecessary(result)
             {
-                // Note that this is a best effort solution - lazily loaded modules won't have a chance to
-                // consume platform event.
-                if (result) {
-                    if (event)
-                        event.consume(true);
+                if (result)
                     return;
-                }
                 processActionIdsSequentially.call(this);
             }
         }
 
         /**
          * @return {boolean}
          */
         function isPossiblyInputKey()
         {
-            if (!event || !WebInspector.isBeingEdited(/** @type {!Node} */ (event.target)) || /^F\d+|Control|Shift|Alt|Meta|Win|U\+001B$/.test(keyIdentifier))
+            if (!event || !WebInspector.isEditing() || /^F\d+|Control|Shift|Alt|Meta|Win|U\+001B$/.test(keyIdentifier))
                 return false;
 
             if (!keyModifiers)
                 return true;
 
             var modifiers = WebInspector.KeyboardShortcut.Modifiers;
             if ((keyModifiers & (modifiers.Ctrl | modifiers.Alt)) === (modifiers.Ctrl | modifiers.Alt))
                 return WebInspector.isWin();
 
             return !hasModifier(modifiers.Ctrl) && !hasModifier(modifiers.Alt) && !hasModifier(modifiers.Meta);
@@ skipping 77 matching lines @@
  * @constructor
  */
 WebInspector.ShortcutRegistry.ForwardedShortcut = function()
 {
 }
 
 WebInspector.ShortcutRegistry.ForwardedShortcut.instance = new WebInspector.ShortcutRegistry.ForwardedShortcut();
 
 /** @type {!WebInspector.ShortcutRegistry} */
 WebInspector.shortcutRegistry;