Bug 265565

Hi Team,

While looking into WebSockets.idl file in reference to web-spec, I noticed few things:

Web-Socket: https://websockets.spec.whatwg.org/#the-websocket-interface

1) Following constructor should be as per Web-Spec:

Current:

constructor(USVString url, optional sequence<DOMString> protocols = []);

From Web-Spec:

constructor(USVString url, optional (DOMString or sequence<DOMString>) protocols = []);

NOTE: If we have second constructor to separate 'DOMString' and 'sequence<DOMString>' but then we have below issue.

> Constructor with 'protocol' while I think it should be 'protocols'. It has missing 's'. I checked 'Gecko' (Websocket.webidl) and didn't find constructor with 'protocol'.

2) 'bufferedAmount' should be 'long long' rather than just 'long':

Current:

readonly attribute unsigned long bufferedAmount;

From Web-Spec:

readonly attribute unsigned long long bufferedAmount;

3) In 'messaging' for 'send()', we have it as 'ArrayBuffer' rather than using typedef 'BufferSource' (looks cleaner IMO):

typedef (ArrayBufferView or ArrayBuffer) BufferSource;

and then:

undefined send((BufferSource or Blob or USVString) data);

rather than current four line:

    undefined send(ArrayBuffer data);
    undefined send(ArrayBufferView data);
    undefined send(Blob data);
    undefined send(USVString data);

__

I just wanted to raise so we can track all of them. I might not be able to fix this quite easily. So I will leave it for others to figure out.

Thanks!