Make MediaStreamGraph's AppendMessage() functionality visible so other callers can interact with the MediaStreamGraph queue

RESOLVED WORKSFORME

Status

()

Core
Audio/Video
RESOLVED WORKSFORME
6 years ago
5 years ago

People

(Reporter: jesup, Assigned: jesup)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [WebRTC] [blocking-webrtc-])

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
Needed for some of the interactions with Pipelines to know when other control messages have been processed (like AddListener/RemoveListener), per ekr
(Assignee)

Comment 1

6 years ago
Created attachment 694463 [details] [diff] [review]
Make AppendMessage (and ControlMessage class) visible on the MediaStreamGraph
(Assignee)

Comment 2

5 years ago
blocking- since we now have NotifyRemoved; might still have some use for other things to synchronize with MSG actions
Whiteboard: [WebRTC] [blocking-webrtc+] → [WebRTC] [blocking-webrtc-]
(Assignee)

Comment 3

5 years ago
No longer needed since roc split MediaStreamGraphImpl.h out; you can run arbitrary code in the MSG command queue via ControlMessage (see AudioNodeStream::SetStreamTimeParameter() in content/media/AudioNodeStream.cpp for an example).
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.