[HTML5] Meta charset sniffing fails with newlines in the attribute value

RESOLVED FIXED

Status

()

defect
P3
normal
RESOLVED FIXED
10 years ago
9 years ago

People

(Reporter: hsivonen, Assigned: hsivonen)

Tracking

Other Branch
Points:
---
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

()

Attachments

(1 attachment, 1 obsolete attachment)

Assignee

Description

10 years ago
<meta charset="
ISO-8859-9
">

Should be ISO-8859-9 but isn't.
Assignee

Updated

10 years ago
Priority: -- → P3
Assignee

Updated

9 years ago
Assignee: nobody → hsivonen
Status: NEW → ASSIGNED
Assignee

Comment 1

9 years ago
Posted patch Trim charset names before use (obsolete) — Splinter Review
Assignee

Updated

9 years ago
Attachment #494360 - Flags: review?(jonas)
Assignee

Comment 3

9 years ago
Testing IE shows that the trimming indeed belongs in the HTML parser--not in the charset alias service--because IE doesn't trim spaced from CSS @charset.
Assignee

Comment 4

9 years ago
Comment on attachment 494360 [details] [diff] [review]
Trim charset names before use, with test hg added this time

Nominating for approval. This is a very simple and low-risk spec correctness fix that implements behavior pioneered by IE.
Attachment #494360 - Flags: approval2.0?
Attachment #494360 - Flags: approval2.0? → approval2.0+
Assignee

Comment 5

9 years ago
http://hg.mozilla.org/mozilla-central/rev/6b2bd606ed82
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Assignee

Updated

9 years ago
Flags: in-testsuite+
You need to log in before you can comment on or make changes to this bug.