Closed Bug 643923 Opened 14 years ago Closed 14 years ago

Change in handling of <noscript>

Categories

(Core :: DOM: HTML Parser, defect)

x86
Windows Vista
defect
Not set
normal

Tracking

()

RESOLVED INVALID

People

(Reporter: billlalonde, Unassigned)

References

Details

(Whiteboard: [due to html5 parser])

Attachments

(1 file)

User-Agent:       Mozilla/5.0 (Windows NT 6.0; rv:2.0) Gecko/20100101 Firefox/4.0
Build Identifier: Mozilla/5.0 (Windows NT 6.0; rv:2.0) Gecko/20100101 Firefox/4.0

Page with the following HTML:

<i>Note: JavaScript is <blink>
<noscript>
</blink>not
</noscript>
enabled
<noscript>
<blink>
</noscript>
.</blink>

now blinks text "not enabled" when scripting is disabled. Previously, only
the period blinked with scripting disabled. Text "enabled." blinks when 
scripting enabled as expected.

Reproducible: Always

Steps to Reproduce:
1. Disable scripting.
2. Display page with specified HTML.
Actual Results:  
text "not enabled" blinks.

Expected Results:  
text "not enabled" should not blink.

I am not sure what other problems might be caused by this issue since <noscript> tag does not appear to be working correctly.
Fx4 introduced a new HTML5 compatible parser, that changed this behaviour.
Previously your code had different results with
<blink> vs. <span style=text-decoration:blink>

The new behaviour is consistent and I strongly assume it's what HTML5 spec requires for badly nested tags.

-> INVALID
Status: UNCONFIRMED → RESOLVED
Closed: 14 years ago
Component: General → HTML: Parser
Product: Firefox → Core
QA Contact: general → parser
Resolution: --- → INVALID
Version: unspecified → Trunk
Attached file Testcase from comment 0 (deleted) —
Ah ok comment 1 beat me to it ;-)

Was about to say toggling html5.parser.enable to false makes the testcase behave like it did before, so due to the new html 5 parser.
Whiteboard: [due to html5 parser]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: