Fix the AudioContext's sample-rate if privacy.resistFingerprinting is enabled
Categories
(Core :: Web Audio, enhancement, P3)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox72 | --- | fixed |
People
(Reporter: padenot, Assigned: padenot)
References
(Blocks 1 open bug)
Details
(Whiteboard: [fingerprinting])
Attachments
(1 file)
Before doing this (the patch is rather simple), I need to make sure this does not leak through enumerateDevice or something like this.
This might have performance implications on Android, as the audio output low-latency path only works with the primary sample-rate of the device. I'll investigate this as well.
44100 or 48000 are both the most common rate (on various OS/hw configuration), I think picking 44100 is good enough on desktop.
On mobile, sometimes 48000 instead of 44100 will bring a nice perf boost, and is super common enough to pick.
|
||
Updated•5 years ago
|
|
Assignee | |
Comment 1•5 years ago
|
||
jib, would you have info about the first sentence? What does change for enumerateDevices if RFP is enabled? Can we find the system default device using one of those mediaDevices methods?
|
||
Updated•5 years ago
|
|
|
||
Comment 2•5 years ago
|
||
enumerateDevices already lies when privacy.resistFingerprinting is enabled (bug 1372073), returning dummy devices.
Additionally, we don't expose capabilities yet anyway through enumerateDevices (bug 1179084), so we're good there.
We don't expose sampleRate yet in track.getSettings() because we don't implement the sampleRate constraint yet bug 1388586.
I'll add notes in the open bugs to remember to respect the privacy.resistFingerprinting pref.
|
||
Updated•5 years ago
|
|
|
Assignee | |
Comment 3•5 years ago
|
||
If the sample-rate is 44100 or 48000, use that, it's common enough (roughly
49.5% each). Else, use 48000.
Pushed by padenot@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/c5dbc172388f Fix the AudioContext's sample-rate if privacy.resistFingerprinting is enabled. r=tjr
Pushed by rmaries@mozilla.com: https://hg.mozilla.org/mozilla-central/rev/c57ebfe0e074 Fix the AudioContext's sample-rate if privacy.resistFingerprinting is enabled. r=tjr
|
||
Comment 6•5 years ago
|
||
| bugherder | ||
|
||
Updated•5 years ago
|
Description
•