Commit 65e85fac authored by Gijs Kruitbosch's avatar Gijs Kruitbosch
Browse files

Bug 1719939 - remove other mentions of browser.proton.enabled, r=jaws

Depends on D134310

Differential Revision: https://phabricator.services.mozilla.com/D134311
parent e7ee5acd
Loading
Loading
Loading
Loading
+3 −2
Original line number Diff line number Diff line
@@ -534,8 +534,9 @@ XPCOMUtils.defineLazyPreferenceGetter(
);

/* Temporary pref while the dust settles around the updated tooltip design
   for tabs and bookmarks toolbar. This will eventually be removed and
   browser.proton.enabled will be used instead. */
   for tabs and bookmarks toolbar. This is a bit of an orphan from the
   proton project. We should figure out what happens with this in
   bug 1746909. */
XPCOMUtils.defineLazyPreferenceGetter(
  this,
  "gProtonPlacesTooltip",
+4 −10
Original line number Diff line number Diff line
@@ -13,7 +13,6 @@
 */

const { Cu } = require("chrome");
const Services = require("Services");
const { LocalizationHelper } = require("devtools/shared/l10n");
const MENUS_L10N = new LocalizationHelper(
  "devtools/client/locales/menus.properties"
@@ -293,15 +292,10 @@ function addTopLevelItems(doc) {
  menu.insertBefore(pageSourceMenu, extensionsForDevelopersMenu);

  const taskManagerMenu = doc.getElementById("menu_taskManager");
  if (Services.prefs.getBoolPref("browser.proton.enabled", false)) {
  const remoteDebuggingMenu = doc.getElementById(
    "menu_devtools_remotedebugging"
  );
  menu.insertBefore(taskManagerMenu, remoteDebuggingMenu);
  } else {
    // When proton is preffed off, this is in the "more" section instead.
    taskManagerMenu.hidden = true;
  }
}

/**
+1 −10
Original line number Diff line number Diff line
@@ -222,16 +222,7 @@ var Heartbeat = class {
        this.ratingContainer.appendChild(ratingElement);
      }

      if (Services.prefs.getBoolPref("browser.proton.enabled")) {
        // This will append if there aren't any .text-link elements.
      this.notice.buttonContainer.append(this.ratingContainer);
      } else {
        this.notice.messageText.flex = 0;
        this.notice.messageDetails.insertBefore(
          this.ratingContainer,
          this.notice.spacer
        );
      }
    }

    // Let the consumer know the notification was shown.
+2 −1
Original line number Diff line number Diff line
@@ -14,7 +14,8 @@ let { PromptUtils } = ChromeUtils.import(
const AdjustableTitle = {
  _cssSnippet: `
    #titleContainer {
      display: flex; /* Try removing me when browser.proton.enabled goes away. */
      /* This gets display: flex by virtue of being a row in a subdialog, from
        * commonDialog.css . */
      flex-shrink: 0;

      flex-direction: row;