Closed
Bug 659763
Opened 14 years ago
Closed 13 years ago
document.open("text/plain") broken with HTML5 parser
Categories
(Core :: DOM: HTML Parser, defect)
Tracking
()
RESOLVED
FIXED
mozilla10
People
(Reporter: Ms2ger, Assigned: hsivonen)
References
()
Details
Attachments
(2 files, 2 obsolete files)
(deleted),
patch
|
bzbarsky
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
smaug
:
review+
|
Details | Diff | Splinter Review |
And apparently, nobody noticed. Not sure if we want to fix at this point?
Comment 1•14 years ago
|
||
I think we do need to fix this, yes. At least as of a few years ago, this feature was still used in various cases...
How did the html5 parser manage to break this? :(
Reporter | ||
Comment 2•14 years ago
|
||
Assignee | ||
Comment 3•14 years ago
|
||
(In reply to comment #0)
> And apparently, nobody noticed. Not sure if we want to fix at this point?
Chrome doesn't support this, either, but since Opera and IE do, let's fix this.
Comment 4•14 years ago
|
||
Can we add a test too?
Assignee | ||
Comment 5•14 years ago
|
||
(In reply to comment #4)
> Can we add a test too?
Yeah, just ran out of time yesterday.
Attachment #536571 -
Flags: review?(bzbarsky)
Assignee | ||
Comment 6•14 years ago
|
||
Attachment #536571 -
Attachment is obsolete: true
Attachment #536571 -
Flags: review?(bzbarsky)
Attachment #536572 -
Flags: review?(bzbarsky)
Comment 7•14 years ago
|
||
Comment on attachment 536572 [details] [diff] [review]
Reftest, with one more test
r=me
Attachment #536572 -
Flags: review?(bzbarsky) → review+
Assignee | ||
Comment 8•13 years ago
|
||
Attachment #536290 -
Attachment is obsolete: true
Attachment #536290 -
Flags: review?(jonas)
Attachment #567088 -
Flags: review?(Olli.Pettay)
Updated•13 years ago
|
Attachment #567088 -
Flags: review?(Olli.Pettay) → review+
Assignee | ||
Comment 9•13 years ago
|
||
Thanks for the reviews. Landed:
https://hg.mozilla.org/integration/mozilla-inbound/rev/f09b3ca76707
https://hg.mozilla.org/integration/mozilla-inbound/rev/804eb7a2a0c7
Flags: in-testsuite? → in-testsuite+
Target Milestone: --- → mozilla10
Comment 10•13 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/f09b3ca76707
https://hg.mozilla.org/mozilla-central/rev/804eb7a2a0c7
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•