Closed Bug 535499 Opened 15 years ago Closed 15 years ago

[HTML5] document.write('&nbsp') breaks cibc.com with the html5 parser on trunk

Categories

(Core :: DOM: HTML Parser, defect, P1)

x86
macOS
defect

Tracking

()

RESOLVED FIXED

People

(Reporter: jrmuizel, Assigned: hsivonen)

References

Details

Attachments

(3 files, 1 obsolete file)

http://cibc.com/ca/personal.html shows a whole bunch of html as plain text:
"<span id="topNavSites"><a href="/ca/sites.html?src=ca">CIBC Websites</a></span> </div>..."
Summary: cibc.com is broken with the html5 parser on trunk → [HTML5]cibc.com is broken with the html5 parser on trunk
Seems related to the <script> tag at the bottom perhaps...
Attached file Mostly minimized test case (obsolete) —
Looks like it's caused by document.write() causing things to go wrong.
The document.write('&nbsp;') causes us to stop parsing properly
Attachment #418114 - Attachment is obsolete: true
Summary: [HTML5]cibc.com is broken with the html5 parser on trunk → [HTML5]document.write('&nbsp') breaks cibc.com with the html5 parser on trunk
Thank you for taking the time to minimize the test case. This may turn out to be a dupe of bug 533111 but the other bug doesn't have as nice a test case.
Assignee: nobody → hsivonen
Status: NEW → ASSIGNED
Priority: -- → P1
Summary: [HTML5]document.write('&nbsp') breaks cibc.com with the html5 parser on trunk → [HTML5] document.write('&nbsp') breaks cibc.com with the html5 parser on trunk
Attached patch FixSplinter Review
This patch fixes the problem but causes an unnecessary speculation failure. Spun off the useless speculation failure as bug 535530.
http://hg.mozilla.org/mozilla-central/rev/f2903895ad4d
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Blocks: 533111
Should we be adding a test case for this?
Flags: in-testsuite?
You need to log in before you can comment on or make changes to this bug.