Open
Bug 866224
Opened 12 years ago
Updated 2 years ago
Feed back to application and/or source MediaStream the resolution actually needed/used by the encoder
Categories
(Core :: WebRTC, defect, P5)
Core
WebRTC
Tracking
()
NEW
backlog | webrtc/webaudio+ |
People
(Reporter: jesup, Unassigned)
References
Details
Per bug 844177 comment 7 through comment 9, this bug is about reworking how the current CodecConfig setup works in AudioSessionConduit and VideoSessionConduit.
Quote:
I do want to avoid anything possibly heavyweight where it's not needed (and the current ConfigureSendMediaCodec() really isn't designed for what we're talking about, as you mention). What we really need to do is to rearchitect it to interact with the bandwidth controls to dynamically adjust the encoding resolution, to deal with source resolution changes, and to feed back "upstream" to gUM that we're encoding at X by Y, since in many cases the camera will scale for us (or at least scale to something close, we the amount of data copying we do is greatly reduced).
*That's* the redesign we need to do.
Reporter | ||
Comment 1•10 years ago
|
||
This is done, except for the "feeding it back upstream to gUM" part. And arguably that should be intermediated by the App in case it was using the stream for something else as well.
Dynamic resolutions are now being done via the loadadaptation code, and dynamic bandwidth is landed. Repurposing this bug
Summary: Redesign how CodecConfigs work in Audio/VideoConduit and support dynamic resolution control → Feed back to application and/or source MediaStream the resolution actually needed/used by the encoder
Whiteboard: [WebRTC] [blocking-webrtc-]
Reporter | ||
Updated•10 years ago
|
Rank: 45
Priority: -- → P4
Reporter | ||
Updated•10 years ago
|
backlog: --- → webRTC+
Comment 2•7 years ago
|
||
Mass change P4->P5 to align with new Mozilla triage process.
Priority: P4 → P5
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•