Closed Bug 1712598 Opened 4 years ago Closed 3 years ago

Let AudioDecoderInputTrack fill silence to the output if we write less frames than expectation

Categories

(Core :: Audio/Video: MediaStreamGraph, enhancement, P3)

enhancement

Tracking

()

RESOLVED FIXED
90 Branch
Tracking Status
firefox90 --- fixed

People

(Reporter: alwu, Assigned: alwu)

References

(Regressed 2 open bugs)

Details

Attachments

(2 files)

Fork from this.

If there is a gap bewteen the audio that is going to be appended next and the amount of frames that we've appended, then we need to fill silence data in order to keep A/V sync correct.

We did that for the DecodedStream [1] before, but that got removed during the refactoring.

[1] https://searchfox.org/mozilla-central/rev/36181d2c169bafd5e13c534851c8b25d1567cfc3/dom/media/mediasink/DecodedStream.cpp#660-667

Depends on D115749

Blocks: 1517199
Attachment #9223293 - Attachment description: Bug 1712598 - fill silence if detecting a gap in audio. → Bug 1712598 - part1 : fill silence if detecting a gap in audio.

Depends on D115857

Pushed by alwu@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/49489b3d5a3e part1 : fill silence if detecting a gap in audio. r=padenot https://hg.mozilla.org/integration/autoland/rev/97a63d63d219 part2 : add a test for A/V sync. r=padenot
Regressions: 1713397
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 90 Branch
Regressions: 1713410
Regressions: 1731813
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: