Closed Bug 95560 Opened 23 years ago Closed 19 years ago

Return "UTF-8" for nsIPlatformCharset while pass kPlatformCharsetSel_FileName

Categories

(Core :: Internationalization, defect)

PowerPC
macOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: ftang, Assigned: nhottanscp)

References

Details

(Keywords: intl)

Attachments

(1 file)

While calling nsIPlatformCharset with kPlatformCharsetSel_FileName on Macintosh, we currently return the charset name of the system script. Since MacOS X use UTF-8 for all the char* function of BSD file system functions and define the file:/// URL as % escaped UTF-8, we probably should return "UTF-8" on MacOS X instead. see bug 95478 and 95481 for more information.
Attached patch patch, not fully tested yet. (deleted) — Splinter Review
fix spacing, remove tabs from the lines you are adding, test various OS versions if you don't find any problems, r=brade
Summary: Retrun "UTF-8" for nsIPlatformCharset while pass kPlatformCharsetSel_FileName → Retrun "UTF-8" for nsIPlatformCharset while pass kPlatformCharsetSel_FileName
Keywords: intl
QA Contact: andreasb → ylong
Status: NEW → ASSIGNED
This is a very very risky move. Code is simple the change is huge. move it to m95
Target Milestone: --- → mozilla0.9.5
move to m0.9.7
Target Milestone: mozilla0.9.5 → mozilla0.9.7
Blocks: 103669
Depends on: 108000
No longer depends on: 108000
Blocks: 108000
Summary: Retrun "UTF-8" for nsIPlatformCharset while pass kPlatformCharsetSel_FileName → Return "UTF-8" for nsIPlatformCharset while pass kPlatformCharsetSel_FileName
move to 0.9.8
Target Milestone: mozilla0.9.7 → mozilla0.9.8
mass move to m9.9
Target Milestone: mozilla0.9.8 → mozilla0.9.9
give this to nhotta, remove target
Assignee: ftang → nhotta
Status: ASSIGNED → NEW
Target Milestone: mozilla0.9.9 → ---
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla1.2
Target Milestone: mozilla1.2alpha → ---
This has been fixed by bug 199384. -> FIXED
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: