Bug 1297113 - Remove unused debugger shortcut keys;r=jlongster draft
authorJulian Descottes <jdescottes@mozilla.com>
Wed, 14 Sep 2016 16:03:05 +0200
changeset 414494 1de811200d2b90d13f4c98d9c94f212e861d8b45
parent 414493 9a453e8484092df3eb52ff350e3289b339e8c6c7
child 414495 6b1f1677f4eb3c7053e3d954c5e5a670ab39e669
push id29683
push userjdescottes@mozilla.com
push dateFri, 16 Sep 2016 12:53:27 +0000
reviewersjlongster
bugs1297113
milestone51.0a1
Bug 1297113 - Remove unused debugger shortcut keys;r=jlongster MozReview-Commit-ID: 4V0zTqTtSts
devtools/client/debugger/debugger.xul
devtools/client/debugger/panel.js
devtools/client/locales/en-US/debugger.dtd
--- a/devtools/client/debugger/debugger.xul
+++ b/devtools/client/debugger/debugger.xul
@@ -184,42 +184,26 @@
          command="nextSourceCommand"/>
     <key id="prevSourceKey"
          keycode="VK_UP"
          modifiers="accel alt"
          command="prevSourceCommand"/>
     <key id="resumeKey"
          keycode="&debuggerUI.stepping.resume1;"
          command="resumeCommand"/>
-    <key id="resumeKey2"
-         keycode="&debuggerUI.stepping.resume2;"
-         modifiers="accel"
-         command="resumeCommand"/>
     <key id="stepOverKey"
          keycode="&debuggerUI.stepping.stepOver1;"
          command="stepOverCommand"/>
-    <key id="stepOverKey2"
-         keycode="&debuggerUI.stepping.stepOver2;"
-         modifiers="accel"
-         command="stepOverCommand"/>
     <key id="stepInKey"
          keycode="&debuggerUI.stepping.stepIn1;"
          command="stepInCommand"/>
-    <key id="stepInKey2"
-         keycode="&debuggerUI.stepping.stepIn2;"
-         modifiers="accel"
-         command="stepInCommand"/>
     <key id="stepOutKey"
          keycode="&debuggerUI.stepping.stepOut1;"
          modifiers="shift"
          command="stepOutCommand"/>
-    <key id="stepOutKey2"
-         keycode="&debuggerUI.stepping.stepOut2;"
-         modifiers="accel shift"
-         command="stepOutCommand"/>
     <key id="fileSearchKey"
          key="&debuggerUI.searchFile.key;"
          modifiers="accel"
          command="fileSearchCommand"/>
     <key id="fileSearchKey"
          key="&debuggerUI.searchFile.altkey;"
          modifiers="accel"
          command="fileSearchCommand"/>
--- a/devtools/client/debugger/panel.js
+++ b/devtools/client/debugger/panel.js
@@ -49,19 +49,17 @@ DebuggerPanel.prototype = {
 
     return targetPromise
       .then(() => this._controller.startupDebugger())
       .then(() => this._controller.connect())
       .then(() => {
         this._toolbox.on("host-changed", this.handleHostChanged);
         // Add keys from this document's keyset to the toolbox, so they
         // can work when the split console is focused.
-        let keysToClone = ["resumeKey", "resumeKey2", "stepOverKey",
-                          "stepOverKey2", "stepInKey", "stepInKey2",
-                          "stepOutKey", "stepOutKey2"];
+        let keysToClone = ["resumeKey", "stepOverKey", "stepInKey", "stepOutKey"];
         for (let key of keysToClone) {
           let elm = this.panelWin.document.getElementById(key);
           this._toolbox.useKeyWithSplitConsole(elm, "jsdebugger");
         }
         this.isReady = true;
         this.emit("ready");
         return this;
       })
--- a/devtools/client/locales/en-US/debugger.dtd
+++ b/devtools/client/locales/en-US/debugger.dtd
@@ -192,23 +192,19 @@
 <!ENTITY debuggerUI.removeAllWatch           "Remove All Watch Expressions">
 <!ENTITY debuggerUI.removeAllWatch.key       "E">
 <!ENTITY debuggerUI.removeAllWatch.accesskey "E">
 
 <!-- LOCALIZATION NOTE (debuggerUI.stepping): These are the keycodes that
   -  control the stepping commands in the debugger (continue, step over,
   -  step in and step out). -->
 <!ENTITY debuggerUI.stepping.resume1    "VK_F8">
-<!ENTITY debuggerUI.stepping.resume2    "VK_SLASH">
 <!ENTITY debuggerUI.stepping.stepOver1  "VK_F10">
-<!ENTITY debuggerUI.stepping.stepOver2  "VK_QUOTE">
 <!ENTITY debuggerUI.stepping.stepIn1    "VK_F11">
-<!ENTITY debuggerUI.stepping.stepIn2    "VK_SEMICOLON">
 <!ENTITY debuggerUI.stepping.stepOut1   "VK_F11">
-<!ENTITY debuggerUI.stepping.stepOut2   "VK_SEMICOLON">
 
 <!-- LOCALIZATION NOTE (debuggerUI.context.newTab):  This is the label
   -  for the Open in New Tab menu item displayed in the context menu of the
   -  debugger sources side menu. This should be the same as
   -  netmonitorUI.context.newTab  -->
 <!ENTITY debuggerUI.context.newTab           "Open in New Tab">
 <!ENTITY debuggerUI.context.newTab.accesskey "O">