Notes, November 25

Den 25. nov. 2015 12:53, skrev Harald Alvestrand:
> Thanks for getting new versions out!
> 
> Overarching items: <crickets>
> 
> 
> Getusermedia PRs:
> 
> #277 Reformat with enforced line wrap at 80 ()
> Let's merge this last.... (or let Dom merge after re-running)
> 
> #278 Fix for Issue 250 ()
> LGTM

Merged

> 
> #279 Fix for Issue 193 ()
> When "framerate" occurs in running text, should we camelCase it or write
> it as "frame rate"? Apart from this, LGTM.

http://darobin.github.io/api-design-cookbook/ recommends interCaps for
frameRate in WebIDL. In running text, we use "frame rate".
Merged after e3dits.

#280 "use remove track algorithm" merged.

#281 needs more time to review

#282 equality for multivalued merged.

#284 overconstrainederror constraint name needs commentary.

> 
> Getusermedia Issues:
> 
> #193 How is adaptive frame rate to be handled? (aboba)
> Fixed by #279
Closed

> #238 Some text on constraints needs updating. (burnburn)



> #242 Management of track members in a stream (aboba)

Closed with #276 reference.

> #244 Explain how extensions are to be done (burnburn)
> #246 track.applyConstraints() doesn't do anything. (burnburn)
> #248 Conflicting information about OverconstrainedError.constrain (burnburn)
> #250 PING: Refer to security-arch on the MUST inside Best Practic (aboba)
> Fixed by #278

Closed

> #257 Non-existent constraints attribute in 4.3.2 (burnburn)
> #267 Permission model for cross-origin iframes (adam-be)
> #268 Iframe sandboxing options for gUM (adam-be)

No proposal yet. Waiting for response from Martin.

> #274 Need description of semantics of multivalued "exact" string  (burnburn)

Closed.

> #276 Move text about firing add/removetrack events from WebRTC 1. (adam-be)
PR #281 exist.

#283 Active state on streams inconsistent

Easiest solution is removing the events.
> 
> 
> 
> WebRTC-PC PRs:
> 
> #317 Make RTCDataChannels Transferable ()
> Still waiting for EKR. Any luck with pinging?

Stefan pinged in PR - no response.

> 
> #349 Defining RTCIceCandidateInformation, which inherits from RTC
> (elagerway)
> #350 RTCRtpSender support objects -> interfaces (dontcallmedom)
> Seems to still be under discussion (both).

Adam-be thinks we have to go dictionary -> interface. Still thinking.
Erik will comment and close.

> #353 Plan X: Add an API for using RID to do simulcast (pthatcherg)
> LGTM - merge?

Merged.
> 
> #365 Adding an accessor for the browser-configured ICE servers ()
> LGTM - merge?

Merged.
> 
> #380 Another bunch of editorial fixes (adam-be)
> Adam - merge? (should likely be merged first, before it goes out of date)

Dom should rebase and merge.

> 
> #397 Make degradationPreference per-sender instead of per-encodin ()
> LGTM. Merge?

Needs rebase. Bernard can merge once it's rebased.
> 
> #398 Make RtpTransceiver.mid nullable and remove RtpSender.mid an ()
> LGTM. Merge?

Merged.
> 
> #401 Fix for Issue 267 ()
> Bitrate definition. Is it OK to skip the "averaged over <time>" part?

TODO needs removal, Bernard will then merge.
> 
> #402 Fix for Issue 391 ()
> Remove DTMF A-D - LGTM

Merged.
> 
> #403 Fix for Issue 377 ()
> audioLevel - enough, or add more text?

Updated. Merged.

#404 Remove "events on mediastream"
Will be dealt with together with mediacapture-main#281

#405 addtrack event note
Also waiting for mediacapture-main#281.

> 
> WebRTC-PC issues
> 
> We need owners from #328 on up.
> Need report from owners on assigned ones.
> 
> #8 Need to add public to RTCIceTransports (alvestrand)
> #133 Streams that become isolated generate errors on PC ()
> #140 DataChannel.onerror callback needs an error argument specifi (burnburn)
> #142 "Hold" unspecified (aboba)
> #150 when closing, all outstanding actions are cancelled and thei (adam-be)
> #151 close is synchronous & idempotent (aboba)
Closed.

> #179 Potentially private IP addresses revealed (alvestrand)
> #224 Not clear when responding PeerConnection reaches iceconnecti
> (alvestrand)
> #227 .mid attribute - random or Null when MID is not present in S (stefhak)
> #230 Add support for WebRTC Data Channel in Workers ()
> Blocked on EKR review of #217
> 
> #244 set{Local|Remote}Description is underspecified (alvestrand)
> #246 RTCIceCandidate's sdpMid and sdpMLineIndex members (alvestrand)
> #253 Assurance that requests to IdP proxy originate from the user
> (martinthomson)
> #257 ICE Candidate should have accessors for protocol-relevant it
> (alvestrand)
> #265 Add Privacy/security review questions ()
> EKR volunteered. Reassign?

Stefan will reach out (again).

> #267 Need a bitrate definition (aboba)
Closed (needs stats PR too)

> #277 Side-effects of mitigations for #179 in demo pages  ()
> Closed
> 
> #286 The "consumer of the session description” conflicts with ter (adam-be)
> #295 Guidance for extending objects vs extending Stats needed (alvestrand)
> #296 Debugging ICE problems needs more info (aboba)
> #305 Describe what happens when media changes (alvestrand)
> #308 RTCRtpSender.get/setParameters is underspecified (adam-be)
> #316 Don't use Interface.attribute to refer to properties of inst (adam-be)
> #319 Work through error definitions (burnburn)
> #328 RTCBundlePolicy Enum section should not be non-normative ()
Bernard

> #329 "none" as a RTCIceCredentialType ()
Bernard, for removing the issue

> #330 Get rid of RTCAnswerOptions? ()
Stefan, since he suggested not closing it.

> #331 Creating an ICE agent is not defined ()
Adam

> #332 Timing of ICE gathering  ()
Adam

> #334 Consider using the streams API for DataChannel instead of ba ()
Closed.

> #335 DOMError is removed from the platform; use DOMException inst ()
Dan

> #336 No definition of "applying an ICE candidate" ()
Adam

> #337 Interfacing between WebRTC spec and JSEP ()
> #338 specify what parts of the SDP can be changed between the cre ()
> #339 Parsing stun/turn URLs ()
> #340 `RTCPriorityType` not aligned with names in rtcweb-transport ()
Closed.

> #344 Should references to RFC5506 and  RFC3550 be normative? ()
> #348 RTCRtpSender section doesn't include the definition for RTCR ()
> #358 Remove mid from sender/receiver ()
> #359 Set a priori codec preferences ()
> #363 m-line match up doesn't work with RtpSender.mid or RtpTransc ()
> #364 Browser calls close() on failed rollback ()
> #369 addTrack's streams parameter is unused. ()
> #370 Add drop option for RTCDegradationPreference  ()
> #373 peerIdentity and certificates are unused in setConfiguration ()
> #375 true as default values for dictionary is bad practice ()
> #377 is `audioLevel` in `RTCRtpContributingSource` positive or ne (aboba)
> #378 `getRemoteCertificates()` is ill-defined ()
> #382 Rejecting unallowed changes to configuration ()
> #383 Keep event handler attribute descriptions simle (don't speci ()
> #384 should "none" be a recognized value in RTCIceTransportPolicy ()
> #386 Certificates expiry? ()
> #388 Section 10.1 (Simple Peer-to-peer Example) should explicitly ()
> #389 Should have a "closed" RTCPeerConnectionState ()
> #390 Ambiguous type of `ssrc` identifier; DOMString or unsigned l ()
> #391 Remove DTMF tones A-D (aboba)
> #392 Language for onnegotiationneeded is incomplete/inconsistent. ()
> #393 Why throw on empty iceServers list? ()
> #395 getRemoteCertificates() should return a sequence of base64 D ()
> #399 RFC5245 says nothing about multiple TURN servers, but WebRTC ()
> #400 STUN checks via TURN servers really should not have peer-ref ()
>

Received on Wednesday, 25 November 2015 16:09:32 UTC