Commit f3251842 authored by brizental's avatar brizental
Browse files

fixup! BB 41919: Letterboxing, add temporarily visible web content-size... 

fixup! BB 41919: Letterboxing, add temporarily visible web content-size indicator on window resizing.

Keep the letterboxing status timeouts in a module-level WeakMap keyed by
window, instead of in a _letterboxingStatusTimeout expando on the chrome
window. Browser chrome tests fail when a test leaves unexpected properties on
the window.
parent 876b282f

toolkit/components/resistfingerprinting/RFPHelper.sys.mjs

+8 −2
Changes for toolkit/components/resistfingerprinting/RFPHelper.sys.mjs: 8 added lines, 2 removed lines.
Original line number Diff line number Diff line
@@ -32,6 +32,9 @@ const kPrefVerticalTabs = "sidebar.verticalTabs"; 


const kPrefResizeWarnings = "privacy.resistFingerprinting.resizeWarnings";



// Timeouts that hide the letterboxing size status, keyed by window.

const letterboxingStatusTimeouts = new WeakMap();



const lazy = {};



ChromeUtils.defineESModuleGetters(lazy, {
@@ -743,8 +746,11 @@ class _RFPHelper { 
            browserParent.classList.add(clazz);

          }

          updateStatus(lastRoundedSize);

          win.clearTimeout(win._letterboxingStatusTimeout);

          win._letterboxingStatusTimeout = win.setTimeout(updateStatus, 1000);

          win.clearTimeout(letterboxingStatusTimeouts.get(win));

          letterboxingStatusTimeouts.set(

            win,

            win.setTimeout(updateStatus, 1000)

          );

        } else {

          updateStatus("");

        }