- From: Stefan Hakansson LK <stefan.lk.hakansson@ericsson.com>
- Date: Tue, 19 Jun 2012 16:19:51 +0200
- To: public-webrtc@w3.org
On 06/19/2012 04:04 PM, Adam Bergkvist wrote: > On 2012-06-18 20:57, Cullen Jennings (fluffy) wrote: >> >> This seems like good proposal, one comment on a small detail. >> >> On Jun 15, 2012, at 1:28 PM, Justin Uberti wrote: >> >>> SessionDescriptionOptions.IncludeAudio = true/false // forces m=audio line to be included >>> SessionDescriptionOptions.IncludeVideo = true/false // forces m=video line to be included >>> SessionDescriptionOptions.UseVoiceActivityDetection = true/false // includes CN codecs if true >> >> I think these three should be constraints, not settings because a given browser may not support any of them. > > Shouldn't useVoiceActivityDetection be on addStream()-level? It seems to > be a more appropriate place to do this since it's more codec > setting/constraint rather than a setting on session-level. It seems more fitting at addStream; however, what about when you add an audio track to a MediaStream that has already been added to PeerConnection? > > /Adam > > >
Received on Tuesday, 19 June 2012 14:20:26 UTC