Re: ACTION-95: Constraints usage in the WebRTC spec

Hi

Since RTCConfiguration is a dictionary (where you can omit keys) it can 
simply be done as:

pc.updateIce({ "iceTransports": newValue });

The other values could then be left unchanged. I anyhow think we need to 
make the configuration, or the components of the configuration, gettable 
from the object when it has been created.

The other question remains. What happens if I do:

pc.updateIce({ "iceServers": totallyNewListOfServers });

/Adam

On 2014-01-21 20:29, Justin Uberti wrote:
> With this API, how would one change just the |iceTransports| param?
> Would you have to remember the last set of iceServers and supply them
> again? Or should we make the current RTCConfiguration gettable from the
> PC object?
>
>
> On Tue, Jan 21, 2014 at 1:41 AM, Adam Bergkvist
> <adam.bergkvist@ericsson.com <mailto:adam.bergkvist@ericsson.com>> wrote:
>
>     On 2013-11-26 15:33, Adam Bergkvist wrote:
>
>         ## IceTransports
>         Used with PeerConnection() Constructor and updateIce()
>
>         Controls which types of candidates the ICE agent is allowed to use.
>
>         The possible values are "none", "relay" and "all". If a browser, for
>         some special reason, is configured to not support all these
>         values, that
>         might be a reason to have it as a constraint.
>
>         Proposal: Move to RTCConfiguration?
>
>
>     updateIce() is changing from
>
>     void updateIce (optional RTCConfiguration configuration, optional
>     MediaConstraints constraints);
>
>     to
>
>     void updateIce (RTCConfiguration configuration);
>
>     Both arguments used to be optional, but unless someone comes up with
>     a reason why you would like to run updateIce() with no arguments,
>     I'll make the first argument mandatory as well.
>
>     To the real question.. Does updateIce() accept changes to everything
>     in the RTCConfiguration dictionary?
>
>     dictionary RTCConfiguration {
>          sequence<RTCIceServer> iceServers;
>          RTCIceTransports       iceTransports = all;
>     }
>
>     /Adam
>
>

Received on Wednesday, 22 January 2014 12:01:51 UTC