643923 - Change in handling of <noscript>

Status: RESOLVED INVALID

(Core :: DOM: HTML Parser, defect)

Description

[bill lalonde]

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.

Comment 1

[j.j.]

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

Comment 2

[Ed Morley [:emorley]]

Attachment: Testcase from comment 0

Comment 3

[Ed Morley [:emorley]]

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.