Microphone permissions request does not display if gUM called from a redirected iframe
Categories
(Firefox :: Site Permissions, defect)
Tracking
()
Tracking | Status | |
---|---|---|
firefox-esr102 | --- | unaffected |
firefox108 | --- | wontfix |
firefox109 | --- | verified |
firefox110 | --- | verified |
People
(Reporter: mitch, Assigned: eemeli)
References
(Regression)
Details
(Keywords: regression)
Attachments
(2 files)
42.23 KB,
image/png
|
Details | |
48 bytes,
text/x-phabricator-request
|
ryanvm
:
approval-mozilla-beta+
|
Details | Review |
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:107.0) Gecko/20100101 Firefox/107.0
Steps to reproduce:
- Build a webpage that calls getUserMedia (gUM) in order to get just the user's microphone.
- Build another webpage (I believe it needs to be on a separate domain/subdomain) which does noting but redirect to the first page.
- Build another webpage which includes an iframe with microphone permissions, and a src linking to the second page.
- Upon loading the third page, expect to see a permissions door hanger asking you to grant access to your microphone.
Actual results:
The permissions door hanger never opens. The gUM promise will stall and never resolve or reject because the user has never responded to the request. A microphone icon is shown in the chrome, where the door hanger should be, but clicking it does nothing.
This can be reproduced by running example 2 in this jsfiddle I created. I used TinyURL to simulate a redirect to an otherwise working example:
https://jsfiddle.net/MitchTalmadge/vnhqdwka/4/
Expected results:
A microphone permissions request door hanger should open, asking the user to allow access to their microphone.
Comment 1•3 years ago
|
||
The Bugbug bot thinks this bug should belong to the 'Firefox::Site Permissions' component, and is moving the bug to that component. Please correct in case you think the bot is wrong.
Comment 2•3 years ago
|
||
Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:108.0) Gecko/20100101 Firefox/108.0
Reproduced the issue in Release 108.0, Beta 109.0b2 and the latest Nightly 110.0a1 (2022-12-14) using Windows 10. This is a regression from Fx107.
Pushlog:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=9fb70b4ae59336b805a1651e7c57c6385cca0717&tochange=fba770dda4d9bf6a7f9fc0c35467a9c45801b120
@Eemeli, could you take a look over this?
Assignee | ||
Comment 3•3 years ago
|
||
Updated•3 years ago
|
Assignee | ||
Comment 4•3 years ago
|
||
Mitch, thank you for reporting thing and providing an excellent reproduction! This turned out to (un)fortunately be a simple typo in the message identifier in the FTL file.
Comment 6•3 years ago
|
||
bugherder |
Comment 7•3 years ago
|
||
The patch landed in nightly and beta is affected.
:eemeli, is this bug important enough to require an uplift?
- If yes, please nominate the patch for beta approval.
- If no, please set
status-firefox109
towontfix
.
For more information, please visit auto_nag documentation.
Assignee | ||
Comment 8•3 years ago
|
||
Comment on attachment 9308889 [details]
Bug 1805319 - Fix typo in webrtc indicator message id. r=flod!
Beta/Release Uplift Approval Request
- User impact if declined: Microphone permissions dialog not shown in certain situations
- Is this code covered by automated tests?: Yes
- Has the fix been verified in Nightly?: Yes
- Needs manual test from QE?: No
- If yes, steps to reproduce:
- List of other uplifts needed: None
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): Very limited fix to identifer in FTL file
- String changes made/needed: A fluent-migrate script is included; no action by localisers is needed
- Is Android affected?: No
Comment 9•3 years ago
|
||
Verified - Fixed in latest Nightly 110.0a1 (2022-12-20) using Windows 10. The microphone permissions request door hanger can be opened without issues.
Comment 10•3 years ago
|
||
Comment on attachment 9308889 [details]
Bug 1805319 - Fix typo in webrtc indicator message id. r=flod!
Approved for 109.0b5, thanks.
Comment 11•3 years ago
|
||
bugherder uplift |
Comment 12•3 years ago
|
||
Verified - Fixed in Beta 109.0b5 (build id: 20221220185745) using Windows 10.
Updated•2 years ago
|
Description
•