[tbb-commits] [torbutton/master] Bug 18980: Remove obsolete toolbar button code

gk at torproject.org gk at torproject.org
Mon May 9 08:13:47 UTC 2016


commit 1e3a5db79447d742bffa5aa773a813f3e379fac9
Author: Arthur Edelstein <arthuredelstein at gmail.com>
Date:   Thu May 5 21:45:29 2016 -0700

    Bug 18980: Remove obsolete toolbar button code
---
 src/chrome/content/torbutton.js | 36 ++++++++----------------------------
 1 file changed, 8 insertions(+), 28 deletions(-)

diff --git a/src/chrome/content/torbutton.js b/src/chrome/content/torbutton.js
index 03a3b56..a8a4ce6 100644
--- a/src/chrome/content/torbutton.js
+++ b/src/chrome/content/torbutton.js
@@ -7,13 +7,6 @@
 // TODO: Double-check there are no strange exploits to defeat:
 //       http://kb.mozillazine.org/Links_to_local_pages_don%27t_work
 
-// TODO: Remove the following HUDService loading code once TBB-ESR24 has
-// been retired.
-if (!window.hasOwnProperty("HUDService")) {
-  XPCOMUtils.defineLazyModuleGetter(this, "HUDService",
-    "resource:///modules/HUDService.jsm");
-}
-
 XPCOMUtils.defineLazyModuleGetter(this, "ConsoleServiceListener",
   "resource://gre/modules/devtools/WebConsoleUtils.jsm");
 XPCOMUtils.defineLazyModuleGetter(this, "WebConsoleUtils",
@@ -610,27 +603,14 @@ function torbutton_init() {
     if (!m_tb_prefs.getBoolPref("extensions.torbutton.inserted_button")) {
       torbutton_log(3, 'Adding button');
       try {
-        if (CustomizableUI) {
-          // ESR31-style toolbar is handled by the existing compiled-in pref.
-          // We also need to prevent first-run toolbar reorg (#13378), so we
-          // reset this toolbar state on first-run.
-          try {
-            m_tb_prefs.clearUserPref("browser.uiCustomization.state");
-          } catch(e) {}
-          CustomizableUI.reset();
-          CustomizableUI.undoReset();
-        } else {
-          // ESR24-style toolbar
-          // TODO: Remove this branch once TBB-ESR24 has been retired.
-          var toolbutton = torbutton_get_button_from_toolbox();
-          var navbar = document.getElementById("nav-bar");
-          // XXX: Will probably fail on fennec. Also explicitly forbidden
-          // by MDC style guides (for good reason). Fix later..
-          var urlbar = document.getElementById("urlbar-container");
-          navbar.insertBefore(toolbutton, urlbar);
-          navbar.setAttribute("currentset", navbar.currentSet);
-          document.persist("nav-bar", "currentset");
-        }
+        // ESR31-style toolbar is handled by the existing compiled-in pref.
+        // We also need to prevent first-run toolbar reorg (#13378), so we
+        // reset this toolbar state on first-run.
+        try {
+          m_tb_prefs.clearUserPref("browser.uiCustomization.state");
+        } catch(e) {}
+        CustomizableUI.reset();
+        CustomizableUI.undoReset();
         torbutton_log(3, 'Button added');
         m_tb_prefs.setBoolPref("extensions.torbutton.inserted_button", true);
       } catch(e) {



More information about the tbb-commits mailing list