Skip to content
Snippets Groups Projects
Verified Commit 39b29d69 authored by henry's avatar henry Committed by Pier Angelo Vendrame
Browse files

Bug 42528: Don't leak system scrollbar size on windows.

parent 168e038e
Branches
No related tags found
No related merge requests found
......@@ -11,6 +11,7 @@
#include "nsLayoutUtils.h"
#include "Theme.h"
#include "nsNativeTheme.h"
#include "nsContentUtils.h"
namespace mozilla::widget {
......@@ -164,7 +165,10 @@ void ScrollbarDrawingWin::RecomputeScrollbarParams() {
}
ConfigureScrollbarSize(defaultSize);
if (StaticPrefs::widget_non_native_theme_win_scrollbar_use_system_size()) {
// Do not leak system size when using ResistFingerprinting.
if (!nsContentUtils::ShouldResistFingerprinting("No context available",
RFPTarget::Unknown) &&
StaticPrefs::widget_non_native_theme_win_scrollbar_use_system_size()) {
ConfigureScrollbarSize(LookAndFeel::GetInt(
LookAndFeel::IntID::SystemScrollbarSize, defaultSize));
}
......
......@@ -11,6 +11,7 @@
#include "nsLayoutUtils.h"
#include "Theme.h"
#include "nsNativeTheme.h"
#include "nsContentUtils.h"
using mozilla::gfx::sRGBColor;
......@@ -352,6 +353,11 @@ bool ScrollbarDrawingWin11::PaintScrollbarThumb(
void ScrollbarDrawingWin11::RecomputeScrollbarParams() {
ScrollbarDrawingWin::RecomputeScrollbarParams();
if (nsContentUtils::ShouldResistFingerprinting("No context available",
RFPTarget::Unknown)) {
// Do not distinguish sizes between windows 10 and 11.
return;
}
// TODO(emilio): Maybe make this configurable? Though this doesn't respect
// classic Windows registry settings, and cocoa overlay scrollbars also don't
// respect the override it seems, so this should be fine.
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment