dtmfsender 1.0 compability

[fippo]

http://w3c.github.io/webrtc-pc/#peer-to-peer-dtmf defines the dtmf sender a bit different than we do here.

It turned out to be fairly trivial to shim (in Edge):

Object.defineProperty(window.RTCRtpSender.prototype, 'dtmf', {
    get: function() {
        if (this._dtmf === undefined) {
            if (this.track.kind === 'audio') {
                this._dtmf = new window.RTCDtmfSender(this);
            } else if (this.track.kind === 'video') {
                this._dtmf = null;
            }
        }
        return this._dtmf;
    }
});

Its a "how do I shim 1.0 ontop of ORTC issue", shall I make a PR for http://draft.ortc.org/#webrtc-compat* ?

[aboba]

Yes, please.

[aboba]

@fippo In the "tip of tree" WebRTC 1.0 spec, dtmf is now always defined if track.kind === 'audio', but is only usable if canInsertDTMF is true, as in ORTC. Can we now close this issue?

[aboba]

I am assuming that this issue has been fixed (in WebRTC 1.0) and can be closed. If not, feel free to re-open it.

[fippo]

oh, i'll update my shim code then. We still need some text, i'll PR now before I forget again ;-)