Bug 1974444: Remove a non-standard attribute, and fix the types on some others. r=mjf,webidl,saschanaz

Differential Revision: https://phabricator.services.mozilla.com/D255375

Author: docfaraday

dom/webidl/RTCDataChannel.webidl

@@ -17,10 +17,9 @@ enum RTCDataChannelType {
 [Exposed=Window]
 interface RTCDataChannel : EventTarget
 {
-  readonly attribute DOMString label;
+  readonly attribute USVString label;
   readonly attribute boolean negotiated;
   readonly attribute boolean ordered;
-  readonly attribute boolean reliable;
   readonly attribute unsigned short? maxPacketLifeTime;
   readonly attribute unsigned short? maxRetransmits;
   readonly attribute USVString protocol;

dom/webidl/RTCPeerConnection.webidl

@@ -63,13 +63,10 @@ dictionary RTCDataChannelInit {
   unsigned short maxPacketLifeTime;
   [EnforceRange]
   unsigned short maxRetransmits;
-  DOMString      protocol = "";
+  USVString      protocol = "";
   boolean        negotiated = false;
   [EnforceRange]
   unsigned short id;
-
-  // These are deprecated due to renaming in the spec, but still supported for Fx53
-  unsigned short maxRetransmitTime;
 };
 
 dictionary RTCOfferAnswerOptions {
@@ -171,7 +168,7 @@ interface RTCPeerConnection : EventTarget  {
 
   readonly attribute RTCSctpTransport? sctp;
   // Data channel.
-  RTCDataChannel createDataChannel (DOMString label,
+  RTCDataChannel createDataChannel (USVString label,
                                     optional RTCDataChannelInit dataChannelDict = {});
   attribute EventHandler ondatachannel;
 };