Closed Bug 88309 Opened 20 years ago Closed 11 years ago
xhtml:script element as :root element does not execute script
The reason is we have checks in several places that assume/check that a script element must have a parent element. I think I have a fix.
Vidur, you broke this with the dynamic script thing, so I think you should review this. I don't know why the parent requirement was there; if you have a reason let me know. The way we cause script execution feels a bit fragile and unintuitive...
Vidur noted that with the first patch we can end up executing scripts twice. The fix 2 should take care of that. But I noticed additional problems. The contents of the script element are visible, if it is the root element. Same is true for title element, and maybe every "hidden" XHTML element if they are used as root elements. That seems kinda weird...
Oh well, I don't like how the fix looks like, and seeing that other XHTML elements have problems working as root moving this to a later milestone. I will try to get the cleanup work in another bug (unused variables etc.)
Target Milestone: mozilla0.9.3 → mozilla1.0
Incidentally, I came up with a workaround for this bug -- using a processing in an XML document (which is itself encapsulated in an HTML iframe). It's very ugly, but it works...
Oh, sorry about that! *sheepish grin*
19 years ago
Target Milestone: mozilla1.0 → mozilla0.9.9
19 years ago
Target Milestone: mozilla0.9.9 → Future
Right click is also disabled when the script element == root element. Will the patch fix this or do I need to file a different bug?
Summary: xhtml:script element as root element does not execute script → xhtml:script element as :root element does not execute script
File a different bug on that, but note the bugnumber here just in case...
Fixed five years ago by bug 286000.
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.