Bug 1422720 - Remove unused functions from globalOverlay.js (goSetMenuValue, goSetAccessKey, goOnEvent, setTooltipText). r?florian draft
authorMark Banner <standard8@mozilla.com>
Tue, 16 Jan 2018 23:16:39 +0000
changeset 721213 4a553dfbdbf343867ffb09212d6ea927b77f0ef2
parent 720739 a3887394965f161d011eebc74e8987a653366e4b
child 746274 6b8dde892e59a6437804e29f5caab00150f717a1
push id95773
push userbmo:standard8@mozilla.com
push dateTue, 16 Jan 2018 23:18:27 +0000
reviewersflorian
bugs1422720
milestone59.0a1
Bug 1422720 - Remove unused functions from globalOverlay.js (goSetMenuValue, goSetAccessKey, goOnEvent, setTooltipText). r?florian MozReview-Commit-ID: IVmDb34wxdi
toolkit/content/globalOverlay.js
--- a/toolkit/content/globalOverlay.js
+++ b/toolkit/content/globalOverlay.js
@@ -95,54 +95,16 @@ function goSetCommandEnabled(aID, aEnabl
   if (node) {
     if (aEnabled)
       node.removeAttribute("disabled");
     else
       node.setAttribute("disabled", "true");
   }
 }
 
-function goSetMenuValue(aCommand, aLabelAttribute) {
-  var commandNode = top.document.getElementById(aCommand);
-  if (commandNode) {
-    var label = commandNode.getAttribute(aLabelAttribute);
-    if (label)
-      commandNode.setAttribute("label", label);
-  }
-}
-
-function goSetAccessKey(aCommand, aValueAttribute) {
-  var commandNode = top.document.getElementById(aCommand);
-  if (commandNode) {
-    var value = commandNode.getAttribute(aValueAttribute);
-    if (value)
-      commandNode.setAttribute("accesskey", value);
-  }
-}
-
-// this function is used to inform all the controllers attached to a node that an event has occurred
-// (e.g. the tree controllers need to be informed of blur events so that they can change some of the
-// menu items back to their default values)
-function goOnEvent(aNode, aEvent) {
-  var numControllers = aNode.controllers.getControllerCount();
-  var controller;
-
-  for (var controllerIndex = 0; controllerIndex < numControllers; controllerIndex++) {
-    controller = aNode.controllers.getControllerAt(controllerIndex);
-    if (controller)
-      controller.onEvent(aEvent);
-  }
-}
-
-function setTooltipText(aID, aTooltipText) {
-  var element = document.getElementById(aID);
-  if (element)
-    element.setAttribute("tooltiptext", aTooltipText);
-}
-
 Object.defineProperty(this, "NS_ASSERT", {
   configurable: true,
   enumerable: true,
   get() {
     delete this.NS_ASSERT;
     var tmpScope = {};
     Components.utils.import("resource://gre/modules/debug.js", tmpScope);
     return this.NS_ASSERT = tmpScope.NS_ASSERT;