Move nsIPlatformCharset from intl/uconv to intl/locale

RESOLVED FIXED in mozilla5

Status

()

defect
RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: smontagu, Assigned: m_kato)

Tracking

Trunk
mozilla5
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments, 1 obsolete attachment)

Reporter

Description

8 years ago
nsIPlatformCharset implementations are the only callers of GetPlatformLocale outside intl/locale. Moving them into intl/locale will it much easier to deCOM intl/locale.
Assignee

Comment 1

8 years ago
Posted patch fix (obsolete) — Splinter Review
Assignee: smontagu → m_kato
Assignee

Updated

8 years ago
Attachment #516835 - Flags: review?(smontagu)
Assignee

Comment 2

8 years ago
Comment on attachment 516835 [details] [diff] [review]
fix

I forgot modify os2 files.
Attachment #516835 - Flags: review?(smontagu)
Assignee

Comment 3

8 years ago
Posted patch fix v2Splinter Review
Attachment #516835 - Attachment is obsolete: true
Attachment #516853 - Flags: review?(smontagu)
Reporter

Updated

8 years ago
Attachment #516853 - Flags: review?(smontagu) → review+
Assignee

Updated

8 years ago
Status: NEW → ASSIGNED
Depends on: post2.0
This needs merging to tip.
Whiteboard: [not-ready-for-cedar]
Keywords: checkin-needed
Whiteboard: [not-ready-for-cedar]
http://hg.mozilla.org/mozilla-central/rev/b6af6b28b729
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Whiteboard: fixed-in-cedar
Target Milestone: --- → mozilla2.2
Why nsICharsetAlias is also moved? It is not used in intl/locale, but in intl/uconv.
Assignee

Comment 9

8 years ago
(In reply to comment #8)
> Why nsICharsetAlias is also moved? It is not used in intl/locale, but in
> intl/uconv.

nsICharsteAlias uses nsUConvPropertySearch.
You need to log in before you can comment on or make changes to this bug.