Verified Commit 453cd063 authored by ma1's avatar ma1
Browse files

fixup! Bug 32308: use direct browser sizing for letterboxing. Bug 30556: align...

fixup! Bug 32308: use direct browser sizing for letterboxing. Bug 30556: align letterboxing with 200x100 new win width stepping

Bug 41259: Do not interfere with devtools responsive mode sizing.
parent 8fded311
Loading
Loading
Loading
Loading
+2 −2
Changes for browser/base/content/browser.css: 2 added lines, 2 removed lines.
Original line number Diff line number Diff line
@@ -140,7 +140,7 @@ body {
  Never modify the following selector without synchronizing
  LETTERBOX_CSS_SELECTOR in RFPHelper.jsm!
**/
.letterboxing .browserStack:not(.exclude-letterboxing) > browser {
.letterboxing .browserContainer:not(.responsive-mode) > .browserStack:not(.exclude-letterboxing) > browser {
  /* width & height to be dynamically set by RFPHelper.jsm */
  outline: 1px solid var(--chrome-content-separator-color);
}
@@ -149,7 +149,7 @@ body {
  outline: initial;
}

:root:not([inDOMFullscreen]) .letterboxing.letterboxing-ready .browserStack:not(.exclude-letterboxing) {
:root:not([inDOMFullscreen]) .letterboxing.letterboxing-ready .browserContainer:not(.responsive-mode) > .browserStack:not(.exclude-letterboxing) {
  place-content: start center;
}

+3 −3
Changes for toolkit/components/resistfingerprinting/RFPHelper.sys.mjs: 3 added lines, 3 removed lines.
Original line number Diff line number Diff line
@@ -437,7 +437,7 @@ class _RFPHelper {
      // preemptively in order to beat race conditions on tab/window creation
      const LETTERBOX_CSS_URL = "chrome://browser/content/browser.css";
      const LETTERBOX_CSS_SELECTOR =
        ".letterboxing .browserStack:not(.exclude-letterboxing) > browser";
        ".letterboxing .browserContainer:not(.responsive-mode) > .browserStack:not(.exclude-letterboxing) > browser";
      for (let ss of document.styleSheets) {
        if (ss.href !== LETTERBOX_CSS_URL) {
          continue;
@@ -557,8 +557,8 @@ class _RFPHelper {

    const roundDimensions = (aWidth, aHeight) => {
      const r = (aWidth, aHeight) => ({
        width: `${aWidth}px`,
        height: `${aHeight}px`,
        width: `var(--rdm-width, ${aWidth}px)`,
        height: `var(--rdm-height, ${aHeight}px)`,
      });

      let result;