DevTools: Don't trigger panel switcher shortcut if alt key is held

third_party/WebKit/Source/devtools/front_end/ui/InspectorView.js

 /*
  * Copyright (C) 2011 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:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 213 matching lines @@
    */
   isDrawerMinimized() {
     return this._drawerSplitWidget.isSidebarMinimized();
   }
 
   /**
    * @param {!Event} event
    */
   _keyDown(event) {
     var keyboardEvent = /** @type {!KeyboardEvent} */ (event);
-    if (!UI.KeyboardShortcut.eventHasCtrlOrMeta(keyboardEvent))
+    if (!UI.KeyboardShortcut.eventHasCtrlOrMeta(keyboardEvent) || event.altKey || event.shiftKey)

[pfeldman, 2017/02/07 22:50:16]

lgtm, but we might need to merge :(

       return;
 
     // Ctrl/Cmd + 1-9 should show corresponding panel.
     var panelShortcutEnabled = Common.moduleSetting('shortcutPanelSwitch').get();
-    if (panelShortcutEnabled && !event.shiftKey && !event.altKey) {
+    if (panelShortcutEnabled) {
       var panelIndex = -1;
       if (event.keyCode > 0x30 && event.keyCode < 0x3A)
         panelIndex = event.keyCode - 0x31;
       else if (
           event.keyCode > 0x60 && event.keyCode < 0x6A &&
           keyboardEvent.location === KeyboardEvent.DOM_KEY_LOCATION_NUMPAD)
         panelIndex = event.keyCode - 0x61;
       if (panelIndex !== -1) {
         var panelName = this._tabbedPane.allTabs()[panelIndex];
         if (panelName) {
@@ skipping 77 matching lines @@
    * @return {boolean}
    */
   handleAction(context, actionId) {
     if (UI.inspectorView.drawerVisible())
       UI.inspectorView._closeDrawer();
     else
       UI.inspectorView._showDrawer(true);
     return true;
   }
 };