Closed Bug 1518834 Opened 2 years ago Closed 2 years ago

Muting locally on appear.in freezes video

Categories

(Core :: Audio/Video: MediaStreamGraph, defect, P1)

defect

Tracking

()

VERIFIED FIXED
mozilla66
Tracking Status
firefox-esr60 --- unaffected
firefox64 --- unaffected
firefox65 + verified
firefox66 + verified

People

(Reporter: drno, Assigned: pehrsons)

References

(Depends on 1 open bug)

Details

(Keywords: regression)

Attachments

(1 file)

When muting the local audio input on appear.in the local preview video freezes as well. Furthermore the video send to the remote side also freezes. If the participant is muted when another party joins the call, the new joiner doesn't get any video until the first party un-mutes him/her-self.

Mozregression run points at bug 1513973.

Assignee: nobody → apehrson
Blocks: 1513973
Keywords: regression
Priority: -- → P1

Andreas can you please have a look?

Flags: needinfo?(apehrson)

It must be the following line:
https://searchfox.org/mozilla-central/rev/5053031ba7621fa8f63f42de4c204ab3561e4e59/dom/media/webrtc/MediaEngineWebRTCAudio.cpp#614
On mute MediaEngineWebRTCMicrophoneSource::Stop is called but we do not want to disable pulling more data.

With pulling disabled we're supposed to still advance the track when data runs out I thought. Or does it only hold true for video?

I'll do the minimal necessary to revert this, and then file a followup to restore and fix this, including a mochitest to make sure we're not regressing it again.

Status: NEW → ASSIGNED
Flags: needinfo?(apehrson)

Try push: https://treeherder.mozilla.org/#/jobs?repo=try&revision=a95e592dc3e1bc2dddb45c08a3d6ad9df17755dd

I also verified the mac build locally for both debug and non-debug.

Pushed by pehrsons@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/6d3e6c915370
Don't disable pulling when stopping microphone source. r=drno

Comment on attachment 9035533 [details]
Bug 1518834 - Don't disable pulling when stopping microphone source. r?drno

[Beta/Release Uplift Approval Request]

Feature/Bug causing the regression: Bug 1513973

User impact if declined: Disabling an audio track will freeze the video track if both tracks come from the same getUserMedia-request.

Is this code covered by automated tests?: No

Has the fix been verified in Nightly?: No

Needs manual test from QE?: Yes

If yes, steps to reproduce: Go to https://appr.tc/r/975803237
Approve the mic and camera request
Click the microphone button to mute your local audio
Expected: video continues
Actual: video freezes

List of other uplifts needed: None

Risk to taking this patch: Low

Why is the change risky/not risky? (and alternatives if risky): Trivial

String changes made/needed:

Attachment #9035533 - Flags: approval-mozilla-beta?
Depends on: 1519015

I've verified Linux build with that fix.

Flags: qe-verify+
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla66

Comment on attachment 9035533 [details]
Bug 1518834 - Don't disable pulling when stopping microphone source. r?drno

[Triage Comment]
Fixes video pausing when the audio is muted under some circumstances. Approved for 65.0b10.

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

I have managed to reproduce this issue using Firefox 66.0a1 (BuildId:20190109214248).

This issue is verified fixed using Firefox 66.0a1 (BuildId:20190111093148) and Firefox 65.0b10 (BuildId:20190110221328) on Windows 10 64bit, macOS 10.13.6 and Ubuntu 18.04 64bit.

The video no longer freezes after muting the local audio input.

Status: RESOLVED → VERIFIED
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.