Skip to content
Snippets Groups Projects
Verified Commit b2f1e697 authored by ma1's avatar ma1 Committed by Pier Angelo Vendrame
Browse files

Bug 41695: Warn on window maximization without letterboxing in RFPHelper module

parent 503893b7
Branches
Tags
1 merge request!1222Bug 43166: Rebased alpha onto 128.3.0esr
......@@ -417,6 +417,8 @@ pref("privacy.resistFingerprinting.letterboxing.vcenter", true);
pref("privacy.resistFingerprinting.letterboxing.gradient", true);
// tor-browser#41918: should we reuse last window sizes if letterboxing is enabled
pref("privacy.resistFingerprinting.letterboxing.rememberSize", false);
// tor-browser#41695: how many warnings we show if user closes them without restoring the window size
pref("privacy.resistFingerprinting.resizeWarnings", 3);
// tor-browser#33282: new windows start at 1400x900 when there's enough screen space, otherwise down by 200x100 blocks
pref("privacy.window.maxInnerWidth", 1400);
pref("privacy.window.maxInnerHeight", 900);
......
......@@ -24,6 +24,8 @@ const kPrefLetterboxingDidForceSize =
const kTopicDOMWindowOpened = "domwindowopened";
const kPrefResizeWarnings = "privacy.resistFingerprinting.resizeWarnings";
const lazy = {};
ChromeUtils.defineLazyGetter(lazy, "logConsole", () =>
......@@ -51,6 +53,84 @@ function forEachWindow(callback) {
}
}
async function windowResizeHandler(aEvent) {
if (RFPHelper.letterboxingEnabled || !RFPHelper.rfpEnabled) {
return;
}
if (Services.prefs.getIntPref(kPrefResizeWarnings) <= 0) {
return;
}
const window = aEvent.currentTarget;
// Wait for end of execution queue to ensure we have correct windowState.
await new Promise(resolve => window.setTimeout(resolve, 0));
switch (window.windowState) {
case window.STATE_MAXIMIZED:
case window.STATE_FULLSCREEN:
break;
default:
return;
}
// Do not add another notification if one is already showing.
const kNotificationName = "rfp-window-resize-notification";
let box = window.gNotificationBox;
if (box.getNotificationWithValue(kNotificationName)) {
return;
}
// Rate-limit showing our notification if needed.
if (Date.now() - (windowResizeHandler.timestamp || 0) < 1000) {
return;
}
windowResizeHandler.timestamp = Date.now();
const decreaseWarningsCount = () => {
const currentCount = Services.prefs.getIntPref(kPrefResizeWarnings);
if (currentCount > 0) {
Services.prefs.setIntPref(kPrefResizeWarnings, currentCount - 1);
}
};
const [label, accessKey] = await window.document.l10n.formatValues([
{ id: "basebrowser-rfp-restore-window-size-button-label" },
{ id: "basebrowser-rfp-restore-window-size-button-ak" },
]);
const buttons = [
{
label,
accessKey,
popup: null,
callback() {
// reset notification timer to work-around resize race conditions
windowResizeHandler.timestamp = Date.now();
// restore the original (rounded) size we had stored on window startup
let { _rfpOriginalSize } = window;
window.setTimeout(() => {
window.resizeTo(_rfpOriginalSize.width, _rfpOriginalSize.height);
}, 0);
},
},
];
box.appendNotification(
kNotificationName,
{
label: { "l10n-id": "basebrowser-rfp-maximize-warning-message" },
priority: box.PRIORITY_WARNING_LOW,
eventCallback(event) {
if (event === "dismissed") {
// user manually dismissed the notification
decreaseWarningsCount();
}
},
},
buttons
);
}
class _RFPHelper {
// ============================================================================
// Shared Setup
......@@ -720,6 +800,7 @@ class _RFPHelper {
}
_attachWindow(aWindow) {
aWindow.addEventListener("sizemodechange", windowResizeHandler);
aWindow.gBrowser.addTabsProgressListener(this);
aWindow.addEventListener("TabOpen", this);
const resizeObserver = (aWindow._rfpResizeObserver =
......@@ -751,6 +832,7 @@ class _RFPHelper {
let browser = tab.linkedBrowser;
this._resetContentSize(browser);
}
aWindow.removeEventListener("sizemodechange", windowResizeHandler);
}
_handleDOMWindowOpened(win) {
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please to comment