Closed Bug 600175 Opened 14 years ago Closed 14 years ago

Don't check for noembed, iframe or noframes when running a script

Categories

(Core :: DOM: Core & HTML, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla2.0b8

People

(Reporter: hsivonen, Assigned: Ms2ger)

References

()

Details

(Keywords: dev-doc-complete, html5)

Attachments

(1 file, 1 obsolete file)

See http://www.w3.org/Bugs/Public/show_bug.cgi?id=10522

Remove the use of InNonScriptingContainer() from nsScriptElement.cpp.
So do other browsers run script in those situations?
(In reply to comment #1)
> So do other browsers run script in those situations?

They (Opera 10.62, IE9 beta 1 and 6.0.472.63 beta) do.

http://hsivonen.iki.fi/test/moz/non-scripting-container.html
http://hsivonen.iki.fi/test/moz/non-scripting-container.xhtml
Ok, then we should be good.  Does this need to happen for 2.0, or is it Just A Bug?
Assignee: nobody → Ms2ger
Status: NEW → ASSIGNED
Attachment #485406 - Flags: review?(bzbarsky)
Attachment #485399 - Attachment is obsolete: true
Comment on attachment 485406 [details] [diff] [review]
Run scripts inside noembed, iframe and noframes elements

(In reply to comment #3)
> Ok, then we should be good.  Does this need to happen for 2.0, or is it Just A
> Bug?

I'm not aware of blocker-level impact, but now that a reviewed patch exists and the patch looks low-risk, it would nice to take it.
Attachment #485406 - Flags: approval2.0?
Attachment #485406 - Flags: approval2.0? → approval2.0+
Keywords: checkin-needed
Landed: http://hg.mozilla.org/mozilla-central/rev/61700880b872
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.0b8
Added notes to

https://developer.mozilla.org/en/Firefox_4_for_developers
https://developer.mozilla.org/En/HTML/Element/Script

I'm assuming this is enough documentation for this little quirk.
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: