-
-
Notifications
You must be signed in to change notification settings - Fork 30
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
Add webidl2js‑globals.js
to automate install of [Exposed]
globals
#190
base: main
Are you sure you want to change the base?
Conversation
webidl2js‑globals.js
to handle setup of [Global]swebidl2js‑globals.js
to automate install of [Exposed]
interfaces onto [Global]
s
Perhaps we can be a bit less ambitious, and separate out the parts that add [Exposed] support to WRT an algorithm to figure out the require order, if the dependency graph turns out to be acyclic then topological sorting should work. If it is cyclic then… we’d have to look into other things. |
The dependency order is already enforced to be acyclic in: webidl2js/lib/constructs/interface.js Lines 468 to 478 in 5af205c
|
3853cad
to
4f4fd30
Compare
webidl2js‑globals.js
to automate install of [Exposed]
interfaces onto [Global]
swebidl2js‑globals.js
to automate install of [Exposed]
globals
4f4fd30
to
500cc28
Compare
I've rebased this, but it doesn't look like it's ready for review yet? Interface files call |
@TimothyGu That’s a leftover from before I renamed it to just |
TODO:
Element
goes afterNode
andEventTarget
, but beforeHTMLElement
.Depends on:
[LegacyWindowAlias]
and[NoInterfaceObject]
#191 (merged)[Exposed]
extended attribute #192 (merged)Supersedes and closes #101.
review?(@TimothyGu)