W3C home > Mailing lists > Public > public-webrtc@w3.org > May 2014

Re: renaming updateIce

From: Adam Bergkvist <adam.bergkvist@ericsson.com>
Date: Fri, 2 May 2014 12:40:37 +0200
Message-ID: <53637625.6020606@ericsson.com>
To: cowwoc <cowwoc@bbs.darktech.org>, <public-webrtc@w3.org>
On 2014-05-01 20:15, cowwoc wrote:
> On 30/04/2014 9:52 PM, Jan-Ivar Bruaroey wrote:
>> On 4/30/14 8:14 PM, Justin Uberti wrote:
>>> Understood. If we want setConfiguration to be appropriate, we would
>>> need to change the behavior from update to overwrite.
>>>
>>> so, updateConfiguration SGTM.
>>
>> Reading updateIce it seems like it effectively overwrites all
>> RTConfiguration settings with what's passed in. It is also an
>> operational step in the constructor. That sounds like overwrite to me.
>> What am I missing?
>>
>> .: Jan-Ivar :.
>>
>>
>
> +1
>
> It's confusing to getConfiguration() which contains all the keys, but
> then having to pass that into updateConfiguration() which does not use
> all the keys.
>
> Meaning, either you overwrite the thing using:
>
> var config = pc.getConfiguration();
> config.changeSomething();
> pc.setConfiguration(config);
>
> Or introduce a syntax that allows you to make modifications (like the
> Builder pattern) but then any property that wasn't set should be
> undefined to indicate that it isn't being overwritten. Alternatively,
> allow the user to pass in a map that only defines the keys that are
> being changed.

The intention with the current updateIce() (old name) is to, as the name 
suggests, do an update. This behavior was discussed in the ACTION-95 
thread [1].

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

The above updates single option, and the snippet below, updates 
everything (with is equal to a set).

var config = pc.getConfiguration();
// modify config
pc.updateIce(config);

/Adam

[1] http://lists.w3.org/Archives/Public/public-webrtc/2014Jan/0219.html
Received on Friday, 2 May 2014 10:41:02 UTC

This archive was generated by hypermail 2.3.1 : Monday, 23 October 2017 15:19:40 UTC