Open Bug 1957469 Opened 1 month ago Updated 15 days ago

RFP: chart distortion e.g. on kucoin

Categories

(Core :: DOM: Security, defect, P5)

defect

Tracking

()

People

(Reporter: thorin, Unassigned)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

Attached image example.jpg

from: https://codeberg.org/librewolf/issues/issues/2396

STR

  • site: https://www.kucoin.com/trade/MAJOR-USDT
  • load with out RFP
    • graph is populated, bars are thin, values are normal text size and fully visible etc
  • load with RFP
    • graph is partially populated, x and y axes (plural of axis) are larger, text labels on axes are larger

Note: example is using nightly 138 but I can also reproduce in Mullvad Browser where we still report dPR as 1 and matching media queries (at 100% zoom)

Not sure what part of RFP is causing this (edit: PS: it's not canvas). Worth testing targets if someone has time

I can't reproduce it :/

BUT if anyone would like to quickly find out which RFP target is causing the issue, they can use https://github.com/mozilla-extensions/fingerprinting-protection-debugger/releases/latest. It has a troubleshoot website button that binary "searches" rfp target similar to mozregression.

I can reproduce it, it's WindowDevicePixelRatio.. (It was actually really easy to reproduce because the chart redraws itself every second so you don't even have to reload when doing the troubleshooting.)

but I can also reproduce in Mullvad Browser where we still report dPR as 1

me culpa - we report as 2, what we haven't backported is making it dynamic and aligning matchmedia

it's WindowDevicePixelRatio

indeed

I've floated the idea with piero that we could return 1 or 2 based on the closest real value, so a binary outcome. This should then help mitigate sizing being overly-scaled, but still reduce blurriness (why we moved to 2 in the first place)

The severity field is not set for this bug.
:freddy, could you have a look please?

For more information, please visit BugBot documentation.

Flags: needinfo?(fbraun)
Severity: -- → S3
Flags: needinfo?(fbraun)
Priority: -- → P5
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: