[tbb-commits] [tor-launcher/master] Bug 33906: Fix Tor-Launcher issues for Firefox 75

gk at torproject.org gk at torproject.org
Wed Jun 10 18:57:54 UTC 2020


commit df0350fcaedf1bf859872edfead31814ef23b507
Author: Kathy Brade <brade at pearlcrescent.com>
Date:   Wed Apr 22 15:46:23 2020 -0400

    Bug 33906: Fix Tor-Launcher issues for Firefox 75
    
    The <wizard> and <dialog> elements can no longer be used as root elements.
    Adapt to the use of shadow DOM within the <wizard> implemention.
    Migrate the locale picker from the general.useragent.locale preference to
      intl.locale.requested.
---
 src/chrome/content/localePicker.xhtml            | 10 +--
 src/chrome/content/network-settings-wizard.xhtml | 14 ++--
 src/chrome/content/network-settings.js           | 88 ++++++++++++++----------
 src/chrome/content/network-settings.xhtml        | 13 ++--
 src/chrome/skin/network-settings.css             | 25 ++-----
 5 files changed, 80 insertions(+), 70 deletions(-)

diff --git a/src/chrome/content/localePicker.xhtml b/src/chrome/content/localePicker.xhtml
index 152b54c..38762a0 100644
--- a/src/chrome/content/localePicker.xhtml
+++ b/src/chrome/content/localePicker.xhtml
@@ -1,6 +1,6 @@
 <?xml version="1.0"?>
 <!--
-   - Copyright (c) 2019, The Tor Project, Inc.
+   - Copyright (c) 2020, The Tor Project, Inc.
    - See LICENSE for licensing information.
    - vim: set sw=2 sts=2 ts=8 et syntax=xml:
   -->
@@ -9,15 +9,16 @@
 <?xml-stylesheet href="chrome://torlauncher/skin/network-settings.css"
                  type="text/css"?>
 
-<!DOCTYPE wizard SYSTEM "chrome://torlauncher/locale/network-settings.dtd">
+<!DOCTYPE window SYSTEM "chrome://torlauncher/locale/network-settings.dtd">
 
-<wizard id="TorLauncherLocalePicker"
-        xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul"
+<window xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul"
+        torwizard="true"
         title="&torlauncher.localePicker.title;"
         windowtype="TorLauncher:LocalePicker"
         persist="screenX screenY"
         onload="initLocaleDialog();">
 
+<wizard id="TorLauncherLocalePicker">
   <script type="application/x-javascript"
           src="chrome://torlauncher/content/network-settings.js"/>
 
@@ -37,3 +38,4 @@
     <label id="forAssistance"/>
   </hbox>
 </wizard>
+</window>
diff --git a/src/chrome/content/network-settings-wizard.xhtml b/src/chrome/content/network-settings-wizard.xhtml
index 98dbdd8..7386067 100644
--- a/src/chrome/content/network-settings-wizard.xhtml
+++ b/src/chrome/content/network-settings-wizard.xhtml
@@ -1,5 +1,5 @@
 <!--
-   - Copyright (c) 2019, The Tor Project, Inc.
+   - Copyright (c) 2020, The Tor Project, Inc.
    - See LICENSE for licensing information.
    - vim: set sw=2 sts=2 ts=8 et syntax=xml:
   -->
@@ -8,18 +8,19 @@
 <?xml-stylesheet href="chrome://torlauncher/skin/network-settings.css"
                  type="text/css"?>
 
-<!DOCTYPE wizard SYSTEM "chrome://torlauncher/locale/network-settings.dtd">
+<!DOCTYPE window SYSTEM "chrome://torlauncher/locale/network-settings.dtd">
 
-<wizard id="TorNetworkSettings"
-        xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul"
+<window xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul"
         xmlns:html="http://www.w3.org/1999/xhtml"
-        defaulttitle="&torsettings.wizard.title.default;"
+        torwizard="true"
         windowtype="TorLauncher:NetworkSettings"
         persist="screenX screenY"
-        buttonlabelextra2="&torsettings.copyLog;"
         onload="initDialog();"
         onunload="deinitDialog();">
 
+<wizard id="TorNetworkSettings"
+        defaulttitle="&torsettings.wizard.title.default;"
+        buttonlabelextra2="&torsettings.copyLog;">
   <script type="application/x-javascript"
           src="chrome://torlauncher/content/network-settings.js"/>
 
@@ -129,3 +130,4 @@
   </hbox>
   <panel id="copyLogFeedbackPanel"/>
 </wizard>
+</window>
diff --git a/src/chrome/content/network-settings.js b/src/chrome/content/network-settings.js
index 9692b40..2f89632 100644
--- a/src/chrome/content/network-settings.js
+++ b/src/chrome/content/network-settings.js
@@ -1,4 +1,4 @@
-// Copyright (c) 2019, The Tor Project, Inc.
+// Copyright (c) 2020, The Tor Project, Inc.
 // See LICENSE for licensing information.
 //
 // vim: set sw=2 sts=2 ts=8 et syntax=javascript:
@@ -20,7 +20,7 @@ XPCOMUtils.defineLazyModuleGetter(this, "TorLauncherBridgeDB",
                           "resource://torlauncher/modules/tl-bridgedb.jsm");
 
 const kPrefPromptForLocale = "extensions.torlauncher.prompt_for_locale";
-const kPrefLocale = "general.useragent.locale";
+const kPrefLocale = "intl.locale.requested";
 const kPrefMatchOSLocale = "intl.locale.matchOS";
 
 // The recommended type is listed first in the dropdown menu.
@@ -138,7 +138,7 @@ function initDialogCommon()
   let wizardElem = getWizard();
   let haveWizard = (wizardElem != null);
 
-  let cancelBtn = document.documentElement.getButton("cancel");
+  let cancelBtn = getButton("cancel");
   if (cancelBtn)
   {
     gCancelLabelStr = cancelBtn.label;
@@ -151,6 +151,18 @@ function initDialogCommon()
 
   if (haveWizard)
   {
+    // Apply styles to elements within the wizard element's shadow DOM.
+    let wizardPageBox = wizardElem.shadowRoot.querySelector(".wizard-page-box");
+    if (wizardPageBox)
+    {
+      wizardPageBox.style.padding = "0px";
+      wizardPageBox.style.margin = "14px 20px 18px 20px";
+    }
+
+    let wizardHeader = wizardElem.shadowRoot.querySelector(".wizard-header");
+    if (wizardHeader)
+      wizardHeader.style.display = "none";
+
     // Hide the Tor Browser logo and associated separator element if the
     // TOR_HIDE_BROWSER_LOGO environment variable is set.
     let env = Cc["@mozilla.org/process/environment;1"]
@@ -291,16 +303,16 @@ function initDialog()
     });
 
     // Relabel the accept button to be "Connect"
-    let okBtn = document.documentElement.getButton("accept");
+    let okBtn = getButton("accept");
     if (okBtn)
       okBtn.label = TorLauncherUtil.getLocalizedString("connect");
 
     // Set "Copy Tor Log" label and move it after the Quit (cancel) button.
-    let copyLogBtn = document.documentElement.getButton("extra2");
+    let copyLogBtn = getButton("extra2");
     if (copyLogBtn)
     {
       copyLogBtn.label = wizardElem.getAttribute("buttonlabelextra2");
-      let cancelBtn = document.documentElement.getButton("cancel");
+      let cancelBtn = getButton("cancel");
       if (cancelBtn && TorLauncherUtil.isMac)
         cancelBtn.parentNode.insertBefore(copyLogBtn, cancelBtn.nextSibling);
     }
@@ -312,7 +324,7 @@ function initDialog()
     }
 
     // Use "Connect" as the finish button label (on the last wizard page).
-    let finishBtn = document.documentElement.getButton("finish");
+    let finishBtn = getButton("finish");
     if (finishBtn)
     {
       finishBtn.label = TorLauncherUtil.getLocalizedString("connect");
@@ -320,7 +332,7 @@ function initDialog()
     }
 
     // Set Discard Settings back button label to match the wizard Back button.
-    let wizardBackBtn = document.documentElement.getButton("back");
+    let wizardBackBtn = getButton("back");
     let backBtn = document.getElementById("discardSettingsGoBack");
     if (wizardBackBtn && backBtn)
       backBtn.label = wizardBackBtn.label;
@@ -374,11 +386,18 @@ function initLocaleDialog()
 
   // Replace the finish button's label ("Done") with the next button's
   // label ("Next" or "Continue").
-  let nextBtn = document.documentElement.getButton("next");
-  let doneBtn = document.documentElement.getButton("finish");
+  let nextBtn = getButton("next");
+  let doneBtn = getButton("finish");
   if (nextBtn && doneBtn)
     doneBtn.label = nextBtn.label;
 
+  // The locale picker does not use the "Back" and "Next" buttons; hide them.
+  let backBtn = getButton("back");
+  if (backBtn)
+    backBtn.style.display = "none";
+  if (nextBtn)
+    nextBtn.style.display = "none";
+
   let { AddonManager } = Cu.import("resource://gre/modules/AddonManager.jsm");
   let addonsPromise = AddonManager.getAddonsByTypes(["locale"]);
   addonsPromise.then(aLangPackAddons =>
@@ -432,13 +451,9 @@ function populateLocaleList(aLangPackAddons)
   };
 
   // Retrieve the current locale so we can select it within the list by default.
-  let curLocale;
-  try
-  {
-    let chromeRegSvc = Cc["@mozilla.org/chrome/chrome-registry;1"]
-                         .getService(Ci.nsIXULChromeRegistry);
-    curLocale = chromeRegSvc.getSelectedLocale("global").toLowerCase();
-  } catch (e) {}
+  let curLocale = Services.locale.requestedLocale;
+  if (curLocale)
+    curLocale = curLocale.toLowerCase();
 
   // Build a list of language info objects (language code plus friendly name).
   let foundCurLocale = false;
@@ -485,9 +500,9 @@ function populateLocaleList(aLangPackAddons)
   let localeList = document.getElementById(kLocaleList);
   for (let infoObj of langInfo)
   {
-    let listItem = document.createElement("richlistitem");
+    let listItem = document.createXULElement("richlistitem");
     listItem.setAttribute("value", infoObj.langCode);
-    let label = document.createElement("label");
+    let label = document.createXULElement("label");
     label.value = infoObj.langName;
     listItem.appendChild(label);
     localeList.appendChild(listItem);
@@ -517,7 +532,7 @@ function maxWidthOfContent()
   for (let i = 0; i < buttons.length; ++i)
     showOrHideButton(buttons[i], true, false);
 
-  let btn = document.documentElement.getButton("cancel");
+  let btn = getButton("cancel");
   let btnContainer = btn.parentElement;
 
   const kWarningIconWidth = 20; // skin/warning.png is 16 plus some margin
@@ -991,7 +1006,7 @@ function updateBootstrapProgress(aStatusObj)
     // visibility=hidden instead of XUL hidden=true so that the "For
     // Assistance" text does not move.
     let btnID = getWizard() ? "finish" : "cancel";
-    let btn = document.documentElement.getButton(btnID);
+    let btn = getButton(btnID);
     if (btn)
       btn.style.visibility = "hidden";
     window.setTimeout(function() { close(); }, 250);
@@ -1248,7 +1263,7 @@ function isShowingErrorOverlay()
 
 function showCopyLogButton(aHaveErrorOrWarning)
 {
-  let copyLogBtn = document.documentElement.getButton("extra2");
+  let copyLogBtn = getButton("extra2");
   if (copyLogBtn)
   {
     let haveWizard = (getWizard() != null);
@@ -1272,7 +1287,7 @@ function showCopyLogButton(aHaveErrorOrWarning)
 
 function restoreCopyLogVisibility()
 {
-  let copyLogBtn = document.documentElement.getButton("extra2");
+  let copyLogBtn = getButton("extra2");
   if (!copyLogBtn)
     return;
 
@@ -1288,14 +1303,10 @@ function restoreCopyLogVisibility()
 // Network Settings window.
 function showOrHideDialogButtons(aShow)
 {
-  let buttonContainer = document.getAnonymousElementByAttribute(
-                             document.documentElement, "anonid", "buttons");
-  if (!buttonContainer)
-  {
-    // The wizard uses "Buttons" (capital 'B').
-    buttonContainer = document.getAnonymousElementByAttribute(
-                             document.documentElement, "anonid", "Buttons");
-  }
+  let wizard = getWizard();
+  let buttonContainer = wizard
+                          ? wizard.shadowRoot.querySelector(".wizard-buttons")
+                          : document.querySelector(".dialog-button-box");
 
   if (buttonContainer)
   {
@@ -1328,7 +1339,7 @@ function setButtonAttr(aID, aAttr, aValue)
   if (!aID || !aAttr)
     return null;
 
-  let btn = document.documentElement.getButton(aID);  // dialog buttons
+  let btn = getButton(aID);  // dialog buttons
   if (!btn)
     btn = document.getElementById(aID);               // other buttons
   if (btn)
@@ -1428,7 +1439,7 @@ function showMenuListPlaceholderText(aElemID)
 
 function overrideButtonLabel(aID, aLabel)
 {
-  let btn = document.documentElement.getButton(aID);
+  let btn = getButton(aID);
   if (btn)
   {
     btn.setAttribute("origLabel", btn.label);
@@ -1446,7 +1457,7 @@ function overrideButtonLabelWithKey(aID, aLabelKey)
 
 function restoreButtonLabel(aID)
 {
-  var btn = document.documentElement.getButton(aID);
+  var btn = getButton(aID);
   if (btn)
   {
     var oldLabel = btn.getAttribute("origLabel");
@@ -1641,7 +1652,7 @@ function onCopyLog()
   chSvc.copyString(gProtocolSvc.TorGetLog(countObj));
 
   // Display a feedback popup that fades away after a few seconds.
-  let copyLogBtn = document.documentElement.getButton("extra2");
+  let copyLogBtn = getButton("extra2");
   let panel = document.getElementById(kCopyLogFeedbackPanel);
   if (copyLogBtn && panel)
   {
@@ -2570,6 +2581,13 @@ function getElemValue(aID, aDefaultValue)
 }
 
 
+function getButton(aButtonName)
+{
+  let dialogOrWizard = document.documentElement.firstChild;
+  return dialogOrWizard.getButton(aButtonName);
+}
+
+
 // Hide and show groupbox based on aElem's checked state. aElem may be a
 // checkbox or radio element.
 function toggleElemUI(aElem)
diff --git a/src/chrome/content/network-settings.xhtml b/src/chrome/content/network-settings.xhtml
index faf2dc9..9d2f6b6 100644
--- a/src/chrome/content/network-settings.xhtml
+++ b/src/chrome/content/network-settings.xhtml
@@ -1,6 +1,6 @@
 <?xml version="1.0"?>
 <!--
-   - Copyright (c) 2019, The Tor Project, Inc.
+   - Copyright (c) 2020, The Tor Project, Inc.
    - See LICENSE for licensing information.
    - vim: set sw=2 sts=2 ts=8 et syntax=xml:
   -->
@@ -9,19 +9,19 @@
 <?xml-stylesheet href="chrome://torlauncher/skin/network-settings.css"
                  type="text/css"?>
 
-<!DOCTYPE dialog SYSTEM "chrome://torlauncher/locale/network-settings.dtd">
+<!DOCTYPE window SYSTEM "chrome://torlauncher/locale/network-settings.dtd">
 
-<dialog id="TorNetworkSettings"
-        xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul"
+<window xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul"
         xmlns:html="http://www.w3.org/1999/xhtml"
         title="&torsettings.dialog.title;"
         windowtype="TorLauncher:NetworkSettings"
         persist="screenX screenY"
-        buttons="accept,cancel,extra2"
-        buttonlabelextra2="&torsettings.copyLog;"
         onload="initDialog();"
         onunload="deinitDialog();">
 
+<dialog id="TorNetworkSettings"
+        buttons="accept,cancel,extra2"
+        buttonlabelextra2="&torsettings.copyLog;">
   <script type="application/x-javascript"
           src="chrome://torlauncher/content/network-settings.js"/>
 
@@ -86,3 +86,4 @@
     </vbox>
   </stack>
 </dialog>
+</window>
diff --git a/src/chrome/skin/network-settings.css b/src/chrome/skin/network-settings.css
index c9452d8..f60c5c6 100644
--- a/src/chrome/skin/network-settings.css
+++ b/src/chrome/skin/network-settings.css
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2019, The Tor Project, Inc.
+ * Copyright (c) 2020, The Tor Project, Inc.
  * See LICENSE for licensing information.
  *
  * vim: set sw=2 sts=2 ts=8 et syntax=css:
@@ -7,36 +7,29 @@
 
 @namespace html "http://www.w3.org/1999/xhtml";
 
-dialog {
+window {
   width: 45em;
   height: 41em;
   font: -moz-dialog;
 }
 
-dialog.os-windows {
+window.os-windows {
   width: 49em;
   height: 42em;
 }
 
-wizard {
+window[torwizard="true"] {
   width: 45em;
   height: 38em;
   font: -moz-dialog;
   padding-top: 0px;
 }
 
-wizard.os-windows {
+window[torwizard="true"].os-windows {
   width: 49em;
   height: 44em;
 }
 
-.wizard-page-box {
-  padding: 0px;
-  margin: 14px 20px 18px 20px;
-}
-
-wizard .wizard-header { display: none; }
-
 #forAssistance {
   margin-left: 12px;
   margin-bottom: 6px;
@@ -104,11 +97,6 @@ wizard[tor_hide_browser_logo="true"] .tbb-logo {
   display: none;
 }
 
-wizard#TorLauncherLocalePicker button[dlgtype="back"],
-wizard#TorLauncherLocalePicker button[dlgtype="next"] {
-  display: none;
-}
-
 /* Show menulist placeholder text in gray. */
 #defaultBridgeType:not([value]) .menulist-label,
 #defaultBridgeType[value=""] .menulist-label,
@@ -124,8 +112,7 @@ wizard#TorLauncherLocalePicker button[dlgtype="next"] {
   margin-left: 1.8em;
 }
 
-dialog.os-mac #bridgeList,
-wizard.os-mac #bridgeList {
+window.os-mac #bridgeList {
   font-size: 90%;
 }
 





More information about the tbb-commits mailing list