Devtools: Remove duplicated shortcut for go to member in Sources

One of the two shortcuts for go to member in Sources conflicts with command menu; Cmd+Shift+O can still be used. BUG=589353 Review URL: https://codereview.chromium.org/1822913004 Cr-Commit-Position: refs/heads/master@{#382516}
binaryage · Mar 22, 2016 · 82aee18 · 82aee18
1 parent da43a0c
commit 82aee18
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 2 deletions.
diff --git a/front_end/components/ShortcutsScreen.js b/front_end/components/ShortcutsScreen.js
@@ -418,7 +418,7 @@ WebInspector.ShortcutsScreen.SourcesPanelShortcuts = {
     ],
 
     GoToMember: [
-        WebInspector.KeyboardShortcut.makeDescriptor("p", WebInspector.KeyboardShortcut.Modifiers.CtrlOrMeta | WebInspector.KeyboardShortcut.Modifiers.Shift)
+        WebInspector.KeyboardShortcut.makeDescriptor("o", WebInspector.KeyboardShortcut.Modifiers.CtrlOrMeta | WebInspector.KeyboardShortcut.Modifiers.Shift)
     ],
 
     GoToLine: [

diff --git a/front_end/sources/SourcesView.js b/front_end/sources/SourcesView.js
@@ -107,7 +107,6 @@ WebInspector.SourcesView.prototype = {
         registerShortcut.call(this, WebInspector.ShortcutsScreen.SourcesPanelShortcuts.CloseEditorTab, this._onCloseEditorTab.bind(this));
         registerShortcut.call(this, WebInspector.ShortcutsScreen.SourcesPanelShortcuts.GoToLine, this._showGoToLineDialog.bind(this));
         registerShortcut.call(this, WebInspector.ShortcutsScreen.SourcesPanelShortcuts.GoToMember, this._showOutlineDialog.bind(this));
-        registerShortcut.call(this, [WebInspector.KeyboardShortcut.makeDescriptor("o", WebInspector.KeyboardShortcut.Modifiers.CtrlOrMeta | WebInspector.KeyboardShortcut.Modifiers.Shift)], this._showOutlineDialog.bind(this));
         registerShortcut.call(this, WebInspector.ShortcutsScreen.SourcesPanelShortcuts.ToggleBreakpoint, this._toggleBreakpoint.bind(this));
         registerShortcut.call(this, WebInspector.ShortcutsScreen.SourcesPanelShortcuts.Save, this._save.bind(this));
         registerShortcut.call(this, WebInspector.ShortcutsScreen.SourcesPanelShortcuts.SaveAll, this._saveAll.bind(this));