Bug 1752833 - remove menubar_can_autohide from browser.css r=sfoster

Differential Revision: https://phabricator.services.mozilla.com/D140374

browser/base/content/browser.css

@@ -94,7 +94,6 @@ body {
   }
 }
 
-%ifdef MENUBAR_CAN_AUTOHIDE
 #toolbar-menubar[autohide="true"] {
   overflow: hidden;
 }
@@ -104,13 +103,10 @@ body {
   height: 0 !important;
   appearance: none !important;
 }
-%endif
 
-%ifdef XP_MACOSX
-#toolbar-menubar {
+#toolbar-menubar:not([autohide]) {
   visibility: collapse;
 }
-%endif
 
 panelmultiview {
   -moz-box-align: start;
@@ -291,13 +287,13 @@ toolbar[customizing] #whats-new-menu-button {
   display: none;
 }
 
-
-%ifdef MENUBAR_CAN_AUTOHIDE
 /* Hide the TabsToolbar titlebar controls if the menubar is permanently shown.
  * (That is, if the menu bar doesn't autohide, and we're not in a fullscreen or
  * popup window.) */
-:root:not([chromehidden~="menubar"], [inFullscreen]) #toolbar-menubar:not([autohide=true]) + #TabsToolbar > :is(.titlebar-buttonbox-container, .titlebar-spacer),
-%endif
+:root:not([chromehidden~="menubar"], [inFullscreen]) #toolbar-menubar[autohide="false"] + #TabsToolbar > :is(.titlebar-buttonbox-container, .titlebar-spacer) {
+  display: none;
+}
+
 :root:not([chromemargin], [inFullscreen]) .titlebar-buttonbox-container,
 :root[inFullscreen] .titlebar-spacer,
 :root:not([tabsintitlebar]) .titlebar-spacer {
@@ -330,11 +326,9 @@ toolbar[customizing] #whats-new-menu-button {
   display: none;
 }
 
-%ifdef MENUBAR_CAN_AUTOHIDE
-#toolbar-menubar[autohide=true]:not([inactive]) + #TabsToolbar > .titlebar-buttonbox-container {
+#toolbar-menubar[autohide="true"]:not([inactive]) + #TabsToolbar > .titlebar-buttonbox-container {
   visibility: hidden;
 }
-%endif
 
 :root[tabsintitlebar] .titlebar-buttonbox {
   position: relative;
@@ -470,13 +464,13 @@ toolbarpaletteitem[removable="false"] {
   opacity: 0.5;
 }
 
-%ifndef XP_MACOSX
-toolbarpaletteitem[place="palette"],
-toolbarpaletteitem[place="menu-panel"],
-toolbarpaletteitem[place="toolbar"] {
-  -moz-user-focus: normal;
+@media not (-moz-platform: macos) {
+  toolbarpaletteitem[place="palette"],
+  toolbarpaletteitem[place="menu-panel"],
+  toolbarpaletteitem[place="toolbar"] {
+    -moz-user-focus: normal;
+  }
 }
-%endif
 
 #bookmarks-toolbar-placeholder,
 #bookmarks-toolbar-button,

browser/base/content/browser.js

@@ -1628,6 +1628,18 @@ var gBrowserInit = {
         document.documentElement.setAttribute("sizemode", "maximized");
       }
     }
+    if (AppConstants.MENUBAR_CAN_AUTOHIDE) {
+      const toolbarMenubar = document.getElementById("toolbar-menubar");
+      // set a default value
+      if (!toolbarMenubar.hasAttribute("autohide")) {
+        toolbarMenubar.setAttribute("autohide", true);
+      }
+      toolbarMenubar.setAttribute(
+        "data-l10n-id",
+        "toolbar-context-menu-menu-bar-cmd"
+      );
+      toolbarMenubar.setAttribute("data-l10n-attrs", "toolbarname");
+    }
 
     // Run menubar initialization first, to avoid TabsInTitlebar code picking
     // up mutations from it and causing a reflow.

browser/base/content/navigator-toolbox.inc.xhtml

@@ -16,11 +16,6 @@
              class="browser-toolbar chromeclass-menubar titlebar-color"
              customizable="true"
              mode="icons"
-#ifdef MENUBAR_CAN_AUTOHIDE
-             data-l10n-id="toolbar-context-menu-menu-bar-cmd"
-             data-l10n-attrs="toolbarname"
-             autohide="true"
-#endif
              context="toolbar-context-menu">
       <toolbaritem id="menubar-items" align="center">
 # The entire main menubar is placed into browser-menubar.inc, so that it can be

browser/base/jar.mn

@@ -34,7 +34,7 @@ browser.jar:
         content/browser/aboutTabCrashed.css           (content/aboutTabCrashed.css)
         content/browser/aboutTabCrashed.js            (content/aboutTabCrashed.js)
         content/browser/aboutTabCrashed.xhtml         (content/aboutTabCrashed.xhtml)
-*       content/browser/browser.css                   (content/browser.css)
+        content/browser/browser.css                   (content/browser.css)
         content/browser/browser.js                    (content/browser.js)
 *       content/browser/browser.xhtml                 (content/browser.xhtml)
         content/browser/browser-a11yUtils.js          (content/browser-a11yUtils.js)