Closed
Bug 1882674
Opened 2 years ago
Closed 1 year ago
Remove media.ondevicechange.enabled pref
Categories
(Core :: WebRTC: Audio/Video, task)
Core
WebRTC: Audio/Video
Tracking
()
RESOLVED
FIXED
137 Branch
| Tracking | Status | |
|---|---|---|
| firefox137 | --- | fixed |
People
(Reporter: gregp, Assigned: gzyfqh)
References
(Blocks 1 open bug)
Details
(Keywords: good-first-bug, Whiteboard: [lang=c++])
Attachments
(1 file)
MediaDevices.ondevicechange has been enabled by default for about 8 years now (bug 1308115). Is this pref still useful?
| Reporter | ||
Comment 1•2 years ago
|
||
Keywords: good-first-bug
Whiteboard: [lang=c++]
Comment 2•2 years ago
|
||
It's not useful to turn this pref off, as it would remove API surface that is now considered part of the web platform. The pref additionally does not actually turn off the event firing of the "devicechange" event, so it has not privacy benefit either. Good riddance.
MediaDevices.ondevicechange has been enabled by default for about 8
years, and it looks like we can now remove it.
Updated•1 year ago
|
Assignee: nobody → gzyfqh
Status: NEW → ASSIGNED
Pushed by ealvarez@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/319a0eaef17a
Remove media.ondevicechange.enabled pref r=emilio
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → WORKSFORME
| Reporter | ||
Comment 5•1 year ago
|
||
Sheriffs will close the bug when your code change lands, you don't have to close it yourself 🙂
Status: RESOLVED → REOPENED
Resolution: WORKSFORME → ---
Thank you for your guidance. Currently the patch has landed on the autoland tree. I will wait for the patch to land on the tip tree.
Comment 7•1 year ago
|
||
| bugherder | ||
Status: REOPENED → RESOLVED
Closed: 1 year ago → 1 year ago
status-firefox137:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 137 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•