Re: Spec question: Using settings dictionaries instead of MediaConstraints ([ACTION-50])

Correct. Their usage would be for the case where you didn't have a sending
stream of that type, or for the GetCapabilities option; in either case they
would force the inclusion of the appropriate m= line.

On Mon, Jun 18, 2012 at 8:29 AM, Harald Alvestrand <harald@alvestrand.no>wrote:

> On 06/18/2012 02:13 PM, Adam Bergkvist wrote:
>
>> On 2012-06-15 21:28, Justin Uberti wrote:
>>
>>> At the interim, it was indicated that using MediaConstraints for
>>> non-media PeerConnection methods was probably not the right fit, one
>>> reason being that most options were only relevant for a specific method,
>>> and it would be good to make it clear which options should be passed to
>>> which methods.
>>>
>>> Therefore I propose that we define other settings dictionaries, similar
>>> to MediaConstraints, but named specifically for the methods in which
>>> they will be used.
>>>
>>> This results in new dictionaries IceOptions and
>>> SessionDescriptionOptions, with values as shown below:
>>>
>>> IceOptions.AllowedCandidates = ("none", "relay", "all) // "all", if not
>>> specified
>>>
>>> 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
>>> SessionDescriptionOptions.**RestartIce = true/false // generates offer
>>> with new ufrag/pwd
>>> SessionDescriptionOptions.**GetCapabilities = true/false // generates
>>> "capabilities" offer
>>>
>>> These fit into the existing API on the createOffer/Answer and updateIce
>>> methods:
>>>
>>> *
>>>
>>> [Constructor (IceServers configuration, optional IceOptions iceOptions)]
>>> interface PeerConnection {
>>>    void createOffer(**SessionDescriptionCallback
>>> successCallback, optional PeerConnectionErrorCallback failureCallback,
>>> optional SessionDescriptionOptions options);
>>>    void createAnswer(**SessionDescriptionCallback
>>> successCallback, optional PeerConnectionErrorCallback failureCallback,
>>> optional SessionDescriptionOptions options);
>>>   ...
>>>
>>>    void updateIce(**optionalIceServersconfiguratio**n,
>>> optional IceOptions options);
>>>
>>>
>>> Does this look reasonable?
>>>
>>
>> This is pretty much how I pictured it when we talked about it at the
>> meeting.
>>
>> A question regarding includeAudio/Video: If I add a MediaStream with both
>> audio and video on both sides I don't have to specify these, right?
>>
>
> that's how I interpret the current docs (and the current code), yes.
>
>
>
>