autogenerate bindings for all of the methods and properties of Document

• Component: DOM
• Status: RESOLVED FIXED

[Maciej Stachowiak]

I did all the ones that actually map to Document impl methods, and do not involve types not handled by the bindings generator yet.

[Maciej Stachowiak]

Created attachment 7492 [details]
autogenerate many Document methods and attributes

[Timothy Hatcher]

The diff doesn't contain Document.idl. Please attach or make a new diff.

[Timothy Hatcher]

Comment on attachment 7492 [details]
autogenerate many Document methods and attributes

Document.idl missing.

[Maciej Stachowiak]

Created attachment 7498 [details]
fixed patch

[Eric Seidel (no email)]

Comment on attachment 7498 [details]
fixed patch

So I think this is fine to land as-is.  However, I also think it would be even better to move more of Document to the idl before landing.  I'm going to trust you that all the tests pass.  Now we just have to hope that we have enough testing coverage. :)

[Maciej Stachowiak]

Created attachment 7520 [details]
patch that does all of it, but doesn't work

[Maciej Stachowiak]

Created attachment 7522 [details]
a version that does the whole thing and actually works!

[Anders Carlsson]

Comment on attachment 7522 [details]
a version that does the whole thing and actually works!

r=me

I'd like to see a comment in CodeGeneratorJS.pm about why Document needs a special destructor

[Lucas Forschler]

Mass moving XML DOM bugs to the "DOM" Component.