18485 2012-08-03 23:57:56 +0000 Change DTMF API to be on PeerConnection 2013-06-13 07:39:26 +0000 1 1 1 Unclassified WebRTC Working Group WebRTC API unspecified PC Linux RESOLVED FIXED P2 normal --- 1 harald public-webrtc fluffy public-webrtc stefan.lk.hakansson oldest_to_newest 71837 0 harald 2012-08-03 23:57:56 +0000 Implementing the proposed DTMF API has turned out to be surprisingly complex compared to the importance of the feature; the typing requirements for MediaStream seem to be heavily complicated by it. Tommy Widenflycht has suggested instead that we define calls to make available: pc.canSendDTMF(MediaStreamTrack) pc.sendDTMF(MediaStreamTrack, tones, duration) This has no impact on the definition of MediaStream and allows implementation of sendDTMF without touching anything but PeerConnection. 71927 1 harald 2012-08-08 09:50:21 +0000 based on discussion on the list, there seems to be 3 alternate descriptions of what sendDTMF actually does. I outline them below as text that can be inserted into the description. A) 1) If the track argument to sendDTMF is not an audio track connected to this PeerConnection on an outgoing SSRC where use of RFC 4733 DTMF has been negotiated, throw a <IllegalParameter> exception. 2) Send the tones using RFC 4733 signalling. B) 1) If the track argument to sendDTMF is not an audio track connected to this PeerConnection, throw an <illegalParameter> exception. 2) If sendDTMF is connected to an outgoing SSRC where use of RFC 4377 is negotiated, send the tones using RFC 4377 and return. 3) Insert the corresponding tones into the media stream as if they were coming from the media source. C) 1) If sendDTMF is connected to an outgoing SSRC on this PeerConnection where use of RFC 4377 is negotiated, send the tones using RFC 4377 and return. 2) Insert the corresponding tones into the media stream as if they were coming from the media source. The difference between B) and C) is that B) insists on using the right PC to generate tones. 71928 2 stefan.lk.hakansson 2012-08-08 10:23:07 +0000 (In reply to comment #1) As for alternatives B and C, I think that the tones should not be inserted in the same MediaStream as the outgoing DTMF. The tones are to be used for local feedback, and you would not like to play the other outgoing audio locally. I think we should go for alternative A initially. > based on discussion on the list, there seems to be 3 alternate descriptions of > what sendDTMF actually does. > > I outline them below as text that can be inserted into the description. > > A) > 1) If the track argument to sendDTMF is not an audio track connected to this > PeerConnection on an outgoing SSRC where use of RFC 4733 DTMF has been > negotiated, throw a <IllegalParameter> exception. > 2) Send the tones using RFC 4733 signalling. > > B) > 1) If the track argument to sendDTMF is not an audio track connected to this > PeerConnection, throw an <illegalParameter> exception. > 2) If sendDTMF is connected to an outgoing SSRC where use of RFC 4377 is > negotiated, send the tones using RFC 4377 and return. > 3) Insert the corresponding tones into the media stream as if they were coming > from the media source. > > C) > 1) If sendDTMF is connected to an outgoing SSRC on this PeerConnection where > use of RFC 4377 is negotiated, send the tones using RFC 4377 and return. > 2) Insert the corresponding tones into the media stream as if they were coming > from the media source. > > The difference between B) and C) is that B) insists on using the right PC to > generate tones. 71937 3 stefan.lk.hakansson 2012-08-08 15:16:10 +0000 (In reply to comment #1) > based on discussion on the list, there seems to be 3 alternate descriptions of > what sendDTMF actually does. > > I outline them below as text that can be inserted into the description. > > A) > 1) If the track argument to sendDTMF is not an audio track connected to this > PeerConnection on an outgoing SSRC where use of RFC 4733 DTMF has been > negotiated, throw a <IllegalParameter> exception. Minor note: should not an <NotSupported> exception be thrown? At least if RFC4733 has not been negotiated. Not sure <IllegalParameter> is an allowed exception, at least it is not listed here: http://www.w3.org/TR/domcore/#domexception > 2) Send the tones using RFC 4733 signalling. > > B) > 1) If the track argument to sendDTMF is not an audio track connected to this > PeerConnection, throw an <illegalParameter> exception. > 2) If sendDTMF is connected to an outgoing SSRC where use of RFC 4377 is > negotiated, send the tones using RFC 4377 and return. > 3) Insert the corresponding tones into the media stream as if they were coming > from the media source. > > C) > 1) If sendDTMF is connected to an outgoing SSRC on this PeerConnection where > use of RFC 4377 is negotiated, send the tones using RFC 4377 and return. > 2) Insert the corresponding tones into the media stream as if they were coming > from the media source. > > The difference between B) and C) is that B) insists on using the right PC to > generate tones. 72125 4 fluffy 2012-08-13 14:52:28 +0000 There is active list discussion on this. I think everyone agrees the current spec is not good and needs a change but not clear yet exactly how to fix this. 72188 5 harald 2012-08-15 08:38:15 +0000 Based on further discussion, a need has been identified to play back tones in synchronity with the DTMF signals going out. This functionality removes the need for the ability to play out tones on local streams only, since it solves the use case that was intended for. The interface should then become: pc.canSendDTMF(MediaStreamTrack) pc.sendDTMF(MediaStreamTrack outTrack, tones, duration, optional MediaStreamTrack localTrack) with the handling rule being: 1) If the outTrack argument to sendDTMF is not an audio track connected to this PeerConnection on an outgoing SSRC where use of RFC 4733 DTMF has been negotiated, throw a <ArgumentError?> exception. 2) Simultaneously do the following: - Send the tones using RFC 4733 signalling on the outTrack - If the localTrack is specified, play out an appropriate audio feedback signal using this track, making sure that sound from localTrack does not feed back into outTrack. 76477 6 harald 2012-10-17 08:38:30 +0000 The previous suggestion, with a track on which feedback would be given, gave some pushback - what the feedback should be, whether a track is the appropriate form of feedback, and so on. New suggestion (still not valid IDL): pc.canSendDTMF(MediaStreamTrack) pc.sendDTMF(MediaStreamTrack outTrack, tones, duration, optional callback) with the handling rule being 1) If the outTrack argument to sendDTMF is not an audio track connected to this PeerConnection on an outgoing SSRC where use of RFC 4733 DTMF has been negotiated, throw a <ArgumentError?> exception. 2) Simultaneously do the following: - Send the tones using RFC 4733 signalling on the outTrack - Every time a tone starts and stops, call the callback with either the character of the tone being played or an empty string argument. Then people can do whatever they like with the feedback tone. Is this simple enough to let us close this issue? 76485 7 harald 2012-10-17 09:44:49 +0000 On review of previous comments ... the error thrown should be <NotSupported>, according to Stefan's comment. 89235 8 harald 2013-06-13 07:39:26 +0000 Closed as fixed. API is in the spec, and is implemented in Chrome.