Closed Bug 659763 Opened 9 years ago Closed 8 years ago

document.open("text/plain") broken with HTML5 parser

Categories

(Core :: DOM: HTML Parser, defect)

x86
Linux
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla10

People

(Reporter: Ms2ger, Assigned: hsivonen)

References

()

Details

Attachments

(2 files, 2 obsolete files)

And apparently, nobody noticed. Not sure if we want to fix at this point?
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?  :(
(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.
Assignee: nobody → hsivonen
Status: NEW → ASSIGNED
Attachment #536290 - Flags: review?(jonas)
Depends on: 479959
Flags: in-testsuite?
Can we add a test too?
Attached patch Reftests (obsolete) — Splinter Review
(In reply to comment #4)
> Can we add a test too?

Yeah, just ran out of time yesterday.
Attachment #536571 - Flags: review?(bzbarsky)
Attachment #536571 - Attachment is obsolete: true
Attachment #536571 - Flags: review?(bzbarsky)
Attachment #536572 - Flags: review?(bzbarsky)
Comment on attachment 536572 [details] [diff] [review]
Reftest, with one more test

r=me
Attachment #536572 - Flags: review?(bzbarsky) → review+
Attachment #536290 - Attachment is obsolete: true
Attachment #536290 - Flags: review?(jonas)
Attachment #567088 - Flags: review?(Olli.Pettay)
Attachment #567088 - Flags: review?(Olli.Pettay) → review+
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
You need to log in before you can comment on or make changes to this bug.