Casing errors in firefox/source/intl/uconv/src/charsetalias.properties prevent encoding aliases from working

RESOLVED FIXED

Status

()

defect
RESOLVED FIXED
10 years ago
9 years ago

People

(Reporter: pub-mozilla, Assigned: smontagu)

Tracking

Trunk
Points:
---
Bug Flags:
wanted1.9.2 ?
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Reporter

Description

10 years ago
User-Agent:       Mozilla/5.0 (Macintosh; U; PPC Mac OS X 10_5_8; en-us) AppleWebKit/531.9 (KHTML, like Gecko) Version/4.0.3 Safari/531.9
Build Identifier: 

<http://mxr.mozilla.org/firefox/source/intl/uconv/src/charsetalias.properties> clearly states that the
“key should always be in lower case ascii”, which is not the case for the following:

csIBM866
csIBM850
csIBM852
csIBM855
csIBM857
csIBM862
csIBM864
csMacintosh

As one would expect, these aliases do not work. Converting them to lowercase would presumably fix the problem.

Reproducible: Always

Comment 1

10 years ago
nsCharsetAlias2::GetPreferred does indeed use LowerCaseEqualsASCII to compare the alias. The actual comparison of the charset (in nsCharsetAlias2::Equals) is case-insensitive.
Assignee: nobody → smontagu
Status: UNCONFIRMED → NEW
Component: General → Internationalization
Ever confirmed: true
Product: Firefox → Core
QA Contact: general → i18n
Version: unspecified → Trunk
Assignee

Comment 2

10 years ago
Posted patch PatchSplinter Review
Good catch!
Attachment #396085 - Flags: review?(VYV03354)

Updated

10 years ago
Flags: wanted1.9.2?
Attachment #396085 - Flags: review?(VYV03354) → review+
Assignee

Comment 3

10 years ago
http://hg.mozilla.org/mozilla-central/rev/1be54db4e40c
Status: NEW → RESOLVED
Closed: 10 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Reporter

Updated

9 years ago
Summary: Casing errors in firefox/source/intl/uconv/src/charsetalias.properties prevents encoding aliases from working → Casing errors in firefox/source/intl/uconv/src/charsetalias.properties prevent encoding aliases from working
You need to log in before you can comment on or make changes to this bug.