Re: Summary of desired spec updates (Was: summary of some ongoing discussions)

On 07/16/2012 08:45 PM, Justin Uberti wrote:
> I've tried to summarize all of these changes and the discussions of the
>   past few weeks into a concrete proposal for an updated API. This
> proposal is attached, and is also available at
> https://docs.google.com/document/d/1nfA1ElWed5PPR4nqenEkxNMMLRfzAsh1yMLe8yRcns8/edit#.

Justin, thanks for the input. I appreciate concrete proposals on how to 
solve issues we have identified!

Personally I think what you propose seem to make sense. Some things have 
not been discussed that much yet, but I would be fine if they were 
included in the Editor's draft (which of course can change based on the 
discussions).

The only thing that I immediately think could perhaps have another 
solution is the async callback to setLocal/setRemote. I think what we 
want to accomplish is a way to handle errors (you are not going to do 
anything on the success callback, right?), and perhaps we should have a 
more generic error reporting mechanism. That said, I have no problem 
using the async callback in the Editor's draft (if there is need to 
specify a way for error reporting) for the time being.

Stefan

>
>
> Summary below.
>
> ----------------
>
> *The following changes have been made based on the consensus summarized in
> http://lists.w3.org/Archives/Public/public-webrtc/2012Jun/0267.html,namely:
>
>   * Addition of RTC prefix to WebRTC-specific types to avoid namespace
>     collisions
>   * Addition of async callbacks to indicate result of
>     SetLocalDescription/SetRemoteDescription APIs
>   * New state names and explicit state diagram set for PeerState/IceState
>
>
> In addition, the following changes were made based on discussions from
> the interim meeting and list feedback:
>
>   * Definition added for RTCIceServer and RTCIceServers
>   * Type field moved to SessionDescription object (and removed from
>     setLocalDesc/setRemoteDesc)
>   * Unnecessary callbacks (onconnecting, onopened) removed
>   * createProvisionalAnswer and restartIce parameters removed (now done
>     via constraints)
>   * Addition of "int mediaIndex" parameter to RTCIceCandidate, to
>     provide a simpler matching of IceCandidate to m-line index
>
>
> Other notes:
>
>   * RTCSessionDescription and RTCIceCandidate remain as objects, not
>     dictionaries; since this is the way things currently are, we opted
>     not to change it.
>   * MediaConstraints are used everywhere, instead of settings
>     dictionaries, as it was made fairly clear that we would end up
>     needing the same sort of constraints behavior from settings
>     dictionaries. We do need to indicate in the documentation which
>     constraints are valid for which API calls.
>
> *
>

Received on Tuesday, 17 July 2012 14:39:12 UTC