Closed
Bug 535499
Opened 15 years ago
Closed 15 years ago
[HTML5] document.write(' ') breaks cibc.com with the html5 parser on trunk
Categories
(Core :: DOM: HTML Parser, defect, P1)
Tracking
()
RESOLVED
FIXED
People
(Reporter: jrmuizel, Assigned: hsivonen)
References
Details
Attachments
(3 files, 1 obsolete file)
24.70 KB,
text/html
|
Details | |
210 bytes,
text/html
|
Details | |
729 bytes,
patch
|
Details | Diff | Splinter Review |
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>..."
Updated•15 years ago
|
Blocks: html5-parsing
Summary: cibc.com is broken with the html5 parser on trunk → [HTML5]cibc.com is broken with the html5 parser on trunk
Reporter | ||
Comment 1•15 years ago
|
||
Reporter | ||
Comment 2•15 years ago
|
||
Seems related to the <script> tag at the bottom perhaps...
Reporter | ||
Comment 3•15 years ago
|
||
Looks like it's caused by document.write() causing things to go wrong.
Reporter | ||
Comment 4•15 years ago
|
||
The document.write(' ') causes us to stop parsing properly
Attachment #418114 -
Attachment is obsolete: true
Reporter | ||
Updated•15 years ago
|
Summary: [HTML5]cibc.com is broken with the html5 parser on trunk → [HTML5]document.write(' ') breaks cibc.com with the html5 parser on trunk
Assignee | ||
Comment 5•15 years ago
|
||
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(' ') breaks cibc.com with the html5 parser on trunk → [HTML5] document.write(' ') breaks cibc.com with the html5 parser on trunk
Assignee | ||
Comment 6•15 years ago
|
||
This patch fixes the problem but causes an unnecessary speculation failure. Spun off the useless speculation failure as bug 535530.
Assignee | ||
Comment 7•15 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/f2903895ad4d
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 8•15 years ago
|
||
Should we be adding a test case for this?
Assignee | ||
Updated•15 years ago
|
Flags: in-testsuite?
You need to log in
before you can comment on or make changes to this bug.
Description
•