Bug 9237 – Autogenerate JSHTMLObjectElement and add GetSVGDocument()

• Status: RESOLVED
• Resolution: FIXED
• Component: DOM

[Eric Seidel (no email)]

This patch fixes a few things:
1. Adds "CanHavePluginObject" support to autogeneration
2. Adds "CheckDocumentSecurity" support to autogeneration
3. Autogenerates JSHTMLObjectElement
4. Fixes contentDocument on HTMLObjectElement
5. Adds getSVGDocument() support to HTMLObjectElement

Issues with this patch:
- CanHavePluginObject is not quite 100% implemented.  The put() and callAsFunction() code has not been moved into the autogen script yet.
- contentDocument does not work until the object has a renderer (bug 9201)
- HTMLEmbedElement is not yet autogenerated

[Eric Seidel (no email)]

Created attachment 8667 [details]
incomplete patch (someone will need to finish it)

[David Kilzer (:ddkilzer)]

Eric has been lobbying me to finish this bug.  :)

[Oliver Hunt]

i'll look into .. poor macdome, i'm going to be harassing you for a bit while i get back up to speed

[Eric Seidel (no email)]

This doesn't quite apply cleanly anymore. :(

[Oliver Hunt]

I know :-/
It hasn't applied cleanly for a while, i really need to learn perl properly before i start hacking it back though :(

[Eric Seidel (no email)]

Created attachment 9598 [details]
An updated patch which does not attempt any autobinding

This patch still probably needs more lovin, including some test cases.

[Eric Seidel (no email)]

Created attachment 9704 [details]
patch for review -- no autogeneration attempted, just added GetSVGDocument

[Alexey Proskuryakov]

Comment on attachment 9704 [details]
patch for review -- no autogeneration attempted, just added GetSVGDocument

There are a few ifdefs for SVG_SUPPORT missing, and the ChangeLog should mention the hspace/vspace fix. With that changes, r=me.

[Eric Seidel (no email)]

I'm resolving this fixed, even though it doesn't actually autogenerate anything.  Anders or someone else can open or clone this if they actually want to autogenerate HTMLObjectElement, etc.  getSVGDocument() works now which is what I cared about.