Closed
Bug 644206
Opened 13 years ago
Closed 13 years ago
attributes/test_text.html should be loaded as UTF-8
Categories
(Core :: Disability Access APIs, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: bzbarsky, Assigned: MarcoZ)
References
Details
Attachments
(1 file)
4.57 KB,
patch
|
surkov
:
review+
|
Details | Diff | Splinter Review |
The Russian word in "area7" in this test is encoded in UTF-8, but the test file is apparently loaded as ISO-8859-1 or something by the test. When I changed chrome:// to always be loaded as UTF-8, the test started failing. Just adding a <meta charset="utf-8"> and fixing the test's expected values should fix this problem.
Reporter | ||
Comment 1•13 years ago
|
||
It looks like text/test_words.html might have similar issues, by the way.
Assignee | ||
Comment 2•13 years ago
|
||
Alex, can you take this one, since you know what the result should be? My screen reader doesn't speak Russian, and even if it did, I'm sure I wouldn't know if the values are correct :D Thanks!
Comment 3•13 years ago
|
||
I can do it. Not today though. It should be enough to fix offset. The russian word 'Привет' is 6 characters, while test expects to see 12.
Assignee | ||
Updated•13 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Comment 5•13 years ago
|
||
Attachment #521466 -
Flags: review?(surkov.alexander)
Updated•13 years ago
|
Attachment #521466 -
Flags: review?(surkov.alexander) → review+
Assignee | ||
Comment 6•13 years ago
|
||
Landed on mozilla-central (Mozilla 4.2): http://hg.mozilla.org/mozilla-central/rev/c4e652d029e4
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
•