Support non-LocalMediaStreams as inputs to RTCPeerConnection/AddStream

RESOLVED FIXED in mozilla35

Status

()

Core
WebRTC
P2
normal
RESOLVED FIXED
3 years ago
2 years ago

People

(Reporter: jesup, Assigned: jesup)

Tracking

(Depends on: 1 bug)

Trunk
mozilla35
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: p=1)

(Assignee)

Description

3 years ago
This is a follow-on to bug 884365, which uses direct notification of AppendToTrack for audio to avoid any delay imposed by MediaStream and avoid needing possible resampling to avoid clock drift (see bug 908834 and bug 818822).

If a stream is attached that's not a LocalMediaStream, MediaPipeline should look for normal MediaStream output instead of the direct notifications.  This will enable gUM plus WebAudio (and other scenarios that are similar) to work.  Note that this will have some added latency, and without the resampling bugs fixed may cause an ever-increasing delay or periodic underruns/glitches.
Priority: -- → P2
Target Milestone: --- → mozilla33
Whiteboard: p=1
Target Milestone: mozilla33 → mozilla35
(Assignee)

Comment 1

2 years ago
I believe this is fixed now (see MediaPipeline.cpp:825) if it was ever actually broken.
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.