1805319 - Microphone permissions request does not display if gUM called from a redirected iframe

Status: RESOLVED FIXED

(Firefox :: Site Permissions, defect)

Description

[Mitch Talmadge]

Attachment: Screenshot showing results of running example 2 in the provided jsfiddle.

User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:107.0) Gecko/20100101 Firefox/107.0

Steps to reproduce:

1. Build a webpage that calls getUserMedia (gUM) in order to get just the user's microphone.
2. Build another webpage (I believe it needs to be on a separate domain/subdomain) which does noting but redirect to the first page.
3. Build another webpage which includes an iframe with microphone permissions, and a src linking to the second page.
4. 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

[BugBot [:suhaib / :marco/ :calixte]]

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

[Alin Ilea, Desktop Test Engineering]

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?

Comment 3

[Eemeli Aro [:eemeli]]

Attachment: Bug 1805319 - Fix typo in webrtc indicator message id. r=flod!

Comment 4

[Eemeli Aro [:eemeli]]

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 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/6b1770b08548
Fix typo in webrtc indicator message id. r=flod,fluent-reviewers,Gijs

Comment 6

[Cristian Tuns]

https://hg.mozilla.org/mozilla-central/rev/6b1770b08548

Comment 7

[BugBot [:suhaib / :marco/ :calixte]]

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 to wontfix.

For more information, please visit auto_nag documentation.

Comment 8

[Eemeli Aro [:eemeli]]

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

[Alin Ilea, Desktop Test Engineering]

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

[Ryan VanderMeulen [:RyanVM]]

Comment on attachment 9308889 [details]
Bug 1805319 - Fix typo in webrtc indicator message id. r=flod!

Approved for 109.0b5, thanks.

Comment 11

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-beta/rev/41648a912d0b

Comment 12

[Alin Ilea, Desktop Test Engineering]

Verified - Fixed in Beta 109.0b5 (build id: 20221220185745) using Windows 10.