[webrtc-pc] RTCDTMFToneChangeEventInit.tone is required (#2253)

youennf has just created a new issue for https://github.com/w3c/webrtc-pc:

== RTCDTMFToneChangeEventInit.tone is required ==
As per https://w3c.github.io/webrtc-pc/#dom-rtcdtmftonechangeeventinit, RTCDTMFToneChangeEventInit.tone is mandatory.

As per https://web-platform-tests.live/webrtc/RTCDTMFSender-ontonechange.https.html, Chrome and Firefox treat it as an optional member, which then defaults to the empty string.

Additionally, Chrome requires a RTCDTMFToneChangeEventInit in RTCDTMFToneChangeEvent constructor as per spec.
Firefox makes it optional which allows creating an empty string tone event easily.

https://github.com/w3c/webrtc-pc/issues/789 decided to go with a mandatory attribute.
https://github.com/w3c/webrtc-pc/issues/816 induces that the constructor init is mandatory as well.

Either Chrome, Firefox and WPT test should be updated according the spec, if this is compatible with existing websites.
Or the spec should be updated to better align with implementations.

Please view or discuss this issue at https://github.com/w3c/webrtc-pc/issues/2253 using your GitHub account

Received on Monday, 5 August 2019 01:03:01 UTC