Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove Web IDL typedefs #9930

Closed
foolip opened this issue Apr 19, 2021 · 3 comments
Closed

Remove Web IDL typedefs #9930

foolip opened this issue Apr 19, 2021 · 3 comments
Labels
data:api 🐇 Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API

Comments

@foolip
Copy link
Collaborator

foolip commented Apr 19, 2021

BCD has some data for Web IDL typedefs, namely ConstrainBoolean, ConstrainDOMString, ConstrainDouble, ConstrainULong and DOMHighResTimestamp.

Typedefs in Web IDL are aliases of other types which are invisible to web developers. DOMHighResTimestamp when converted to JavaScript is a Number, but one that has a certain interpretation.

Because typedefs are aliases, they have no compat story of their own separately from the APIs that use the typedefs. In this way they are similar to dictionaries, but even more trivial.

This came up in #9892.

@foolip
Copy link
Collaborator Author

foolip commented Apr 19, 2021

Also worth noting is that MDN documents ConstrainBoolean as a dictionary, but its spec definition is:

typedef (boolean or ConstrainBooleanParameters) ConstrainBoolean;

@queengooborg queengooborg added the data:api 🐇 Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API label Apr 19, 2021
@saschanaz
Copy link
Contributor

Also #6810.

@queengooborg
Copy link
Collaborator

We've now gotten rid of all the typedefs in BCD that we know of! I'm going to close this issue -- let's track any future typedefs we may find in #6810.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
data:api 🐇 Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API
Projects
None yet
Development

No branches or pull requests

3 participants