Draft: draft-ietf-rtcweb-jsep

Commentary: This contains the ice - options changes as well - will take another look once that PR is merged . 
 looks like a straightforward change - lgtm 
 LGTM 
 defines this attribute on media level , while  says it is a session level attribute . I understand that there is development ongoing here ( e.g. 5245bis ) , but given that canTrickleIceCandidates is an API on the PeerConnection it makes more sense IMO to have ice - options : trickle on the session level than media line level . If we want this on media line line level , canTrickleIceCandidates should be a ( readonly ) Transceiver attribute . To that seems messy though , what would happen if  is negotiated but one media line can do trickle but the other not ? the relation between the application of an Answer and canTrickleIceCandidates is spelled out , the same thing should be done for application of an Offer . 
 Decision on 1 ): simplest thing here is to move ice - options to session level , right after " An " a = msid - semantic : WMS " line MUST be ... " 2 ) yes , this should be put into handling a remote description , not handling an answer . 
 I can take this . 
 JSEP requires sending q = msid - semantics : WMS : If I read correctly , that is no longer defined , so that requirement should probably go . I 'm happy to provide a PR that removes the requirement , the example usage and updates the reference , but I assume that updating the reference involves more work than just this , so I 'll refrain from doing so unless I hear it would still be useful . 
 I think we can just remove this line , and scrub it from the examples . More work is probably needed to fix up the refs regarding a = msid , but I think this part should be easy to handle .
Paragraph (version A):
An "a=msid-semantic:WMS" line MUST be added, as specified in I-
D.ietf-mmusic-msid, Section 4.
Paragraph (version A):
Within the data m= section, the "a=mid", "a=ice-ufrag", "a=ice-pwd",
"a=ice-options", "a=fingerprint", and "a=setup" lines MUST be
included as mentioned above, along with an "a=fmtp:webrtc-
datachannel" line and an "a=sctp-port" line referencing the SCTP port
number as defined in I-D.ietf-mmusic-sctp-sdp, Section 4.1.
Paragraph (version B):
Within the data m= section, the "a=mid", "a=ice-ufrag", "a=ice-pwd",
"a=fingerprint", and "a=setup" lines MUST be included as mentioned
above, along with an "a=fmtp:webrtc-datachannel" line and an "a=sctp-
port" line referencing the SCTP port number as defined in I-D.ietf-
mmusic-sctp-sdp, Section 4.1.
Paragraph (version A):
The first step in generating an initial answer is to generate
session-level attributes.  The process here is identical to that
indicated in the Initial Offers section above.
Paragraph (version B):
The first step in generating an initial answer is to generate
session-level attributes.  The process here is identical to that
indicated in the Initial Offers section above, except that the
"a=ice-options" line, with the "trickle" option, as specified in I-
D.ietf-mmusic-trickle-ice, Section 4 is only included if such an
option was present in the offer.
Paragraph (version A):
If the "trickle" ICE option is present in the offer, an "a=ice-
options" line, with the "trickle" option, as specified in I-
D.ietf-mmusic-trickle-ice, Section 4.
Paragraph (version A):
Within the data m= section, the "a=mid", "a=ice-ufrag", "a=ice-pwd",
"a=ice-options", "a=candidate", "a=fingerprint", and "a=setup" lines
MUST be included as mentioned above, along with an "a=fmtp:webrtc-
datachannel" line and an "a=sctp-port" line referencing the SCTP port
number as defined in I-D.ietf-mmusic-sctp-sdp, Section 4.1.
Paragraph (version B):
Within the data m= section, the "a=mid", "a=ice-ufrag", "a=ice-pwd",
"a=candidate", "a=fingerprint", and "a=setup" lines MUST be included
as mentioned above, along with an "a=fmtp:webrtc-datachannel" line
and an "a=sctp-port" line referencing the SCTP port number as defined
in I-D.ietf-mmusic-sctp-sdp, Section 4.1.
Paragraph (version A):
TODO: msid-semantic, identity, rtcp-rsize, rtcp-mux, and any other
attribs valid at session level.
Paragraph (version B):
TODO: identity, rtcp-rsize, rtcp-mux, and any other attribs valid
at session level.