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

Re: Panic between createOffer() and setLocalDescription()

From: <piranna@gmail.com>
Date: Wed, 19 Feb 2014 13:46:10 +0100
Message-ID: <CAKfGGh31fAzNRorBvdRv6s9iA1BmY+Y+mN372hntj6Ta+Be2VA@mail.gmail.com>
To: Iņaki Baz Castillo <ibc@aliax.net>
Cc: public-webrtc <public-webrtc@w3.org>, Stefan Håkansson LK <stefan.lk.hakansson@ericsson.com>, Christer Holmberg <christer.holmberg@ericsson.com>, Silvia Pfeiffer <silviapfeiffer1@gmail.com>
Reviewing the specification of setLocalDescription() and
setRemoteDescription(), I have seen that successCallback and
failureCallback are set as non-nullable, non-optional attributes,
while seems that Chrome and Firefox support both to set null and omit
them, and also I think it makes sense for example if you are waiting
on the onicecandidate event to have fully defined the SDP string
previously to send it to the other peer. I believe the spec should be
changed on this point, independtly that it's decided to move the API
to use Promises in the future or not.

2014-02-19 10:05 GMT+01:00 piranna@gmail.com <piranna@gmail.com>:
>> - It forces the developer to retrieve a SDP from the PeerConnection (via
>> createOffer) and then to pass it back again to the same PeerConnection (via
>> setLocalDescription).
>>
> That's the thing I find most annoying, that it's the same PeerConnection
> object. Wouldn't it be possible that when calling to createOffer() and
> createAnswer() they call internally to setLocalDescription(), so developers
> only need to call it explicitly if they need to modify the SDP string? This
> will make easier the usage of vast mayority of use cases, and also it is a
> backwards compatible modification...



-- 
"Si quieres viajar alrededor del mundo y ser invitado a hablar en un
monton de sitios diferentes, simplemente escribe un sistema operativo
Unix."
- Linus Tordvals, creador del sistema operativo Linux
Received on Wednesday, 19 February 2014 12:46:57 UTC

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