Feature policy should limit what info enumerateDevices() reveals in iframes.
Categories
(Core :: WebRTC: Audio/Video, defect, P2)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox71 | --- | fixed |
People
(Reporter: jib, Assigned: tnguyen)
References
(Blocks 1 open bug)
Details
Attachments
(2 files)
Revealing information on cameras and microphones that cannot be used in getUserMedia make no sense.
The spec says the same. See https://github.com/w3c/mediacapture-main/issues/549
|
||
Updated•5 years ago
|
|
Assignee | |
Updated•5 years ago
|
|
|
Assignee | |
Comment 1•5 years ago
|
||
|
|
Assignee | |
Comment 2•5 years ago
|
||
Pushed by tnguyen@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/044e224f7011 Move IsActivelyCapturingOrHasAPermission out of devices iteration. r=jib https://hg.mozilla.org/integration/autoland/rev/6f9782261d63 enumerateDevices should only expose devices which are allowed to use by FeturePolicy r=baku,jib
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/18873 for changes under testing/web-platform/tests
Upstream web-platform-tests status checks passed, PR will merge once commit reaches central.
|
||
Comment 6•5 years ago
|
||
Backed out 2 changesets (bug 1548087) for wpt failures at audio-output/setSinkId.https.html
Backout: https://hg.mozilla.org/integration/autoland/rev/e10113881513fc187b71dd546db42aebf62ff7a9
Failure push: https://treeherder.mozilla.org/#/jobs?repo=autoland&revision=6f9782261d634349de19f4680a10c7dea59ed4e8
Failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=265170219&repo=autoland&lineNumber=1613
Upstream PR was closed without merging
|
|
Assignee | |
Comment 8•5 years ago
|
||
I don't think I change anything in andoid getSinkID.
In fact I could see the failure appears before my change.
Could you please check again? Thanks
Failures just 1 changeset before my push
https://treeherder.mozilla.org/#/jobs?repo=autoland&selectedJob=265179226&searchStr=android%2C7.0%2Cx86-64%2Cdebug%2Cweb%2Cplatform%2Ctests%2Ctest-android-em-7.0-x86_64%2Fdebug-geckoview-web-platform-tests-e10s-5%2Cw%28wpt5%29&tochange=6f9782261d634349de19f4680a10c7dea59ed4e8&fromchange=787d1f05a316abb2c26995d031a51affb439287b
|
|
Assignee | |
Comment 9•5 years ago
|
||
And after backout, the failure still happens
https://treeherder.mozilla.org/#/jobs?repo=autoland&fromchange=68fad864eca483970fcf0f24ea33b9ebb37b1b0a&selectedJob=265194399
|
||
Comment 10•5 years ago
|
||
Pushed by tnguyen@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/8dd7098c545c Move IsActivelyCapturingOrHasAPermission out of devices iteration. r=jib https://hg.mozilla.org/integration/autoland/rev/7a0863c41c16 enumerateDevices should only expose devices which are allowed to use by FeturePolicy r=baku,jib
|
|
Assignee | |
Updated•5 years ago
|
Upstream web-platform-tests status checks passed, PR will merge once commit reaches central.
|
||
Comment 12•5 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/8dd7098c545c
https://hg.mozilla.org/mozilla-central/rev/7a0863c41c16
Upstream PR merged by moz-wptsync-bot
Description
•