Closed Bug 1805319 Opened 3 years ago Closed 3 years ago

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

Categories

(Firefox :: Site Permissions, defect)

Firefox 107
defect

Tracking

()

RESOLVED FIXED
110 Branch
Tracking Status
firefox-esr102 --- unaffected
firefox108 --- wontfix
firefox109 --- verified
firefox110 --- verified

People

(Reporter: mitch, Assigned: eemeli)

References

(Regression)

Details

(Keywords: regression)

Attachments

(2 files)

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.

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.

Component: Untriaged → Site Permissions

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?

Severity: -- → S3
Status: UNCONFIRMED → NEW
Has STR: --- → yes
Ever confirmed: true
Flags: needinfo?(earo)
Keywords: regression
Regressed by: 1793572
Assignee: nobody → earo
Status: NEW → ASSIGNED

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.

Flags: needinfo?(earo)
Pushed by earo@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/6b1770b08548 Fix typo in webrtc indicator message id. r=flod,fluent-reviewers,Gijs
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 110 Branch

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.

Flags: needinfo?(earo)

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
Flags: needinfo?(earo)
Attachment #9308889 - Flags: approval-mozilla-beta?

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 on attachment 9308889 [details]
Bug 1805319 - Fix typo in webrtc indicator message id. r=flod!

Approved for 109.0b5, thanks.

Attachment #9308889 - Flags: approval-mozilla-beta? → approval-mozilla-beta+

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

You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: