Closed Bug 272199 Opened 20 years ago Closed 20 years ago

Cannot reconvert with Japanist(Japanese IME) on WinNT

Categories

(Core :: Internationalization, defect)

x86
Windows 2000
defect
Not set
major

Tracking

()

RESOLVED FIXED

People

(Reporter: masayuki, Assigned: masayuki)

Details

(Keywords: inputmethod, intl)

Attachments

(1 file, 1 obsolete file)

On NT, we cannot reconvert with Japanist.
Because Japanist need Unicode string, but Mozilla send CP932 string.

On NT, window of Mozilla is Unicode window. Therefore, on unicode window,
we should send unicode string when reconvert.
Attached patch Patch (obsolete) — Splinter Review
I tested with MS-IME2000/ATOK17/Japanist2003 on Win2000/WinME.
Attachment #167323 - Flags: review?(emaijala)
Status: NEW → ASSIGNED
Comment on attachment 167323 [details] [diff] [review]
Patch

How about adding nsToolkit::mIsNT to the calls to OnIMERequest instead of the
default parameter? I just find this a little bit unusual way of using a default
parameter. I'd actually rather remove the default  altogether and add the
parameter to all calls (there aren't many).
Attachment #167323 - Flags: review?(emaijala) → review-
Attached patch PatchSplinter Review
Attachment #167323 - Attachment is obsolete: true
Attachment #168004 - Flags: review?(emaijala)
Comment on attachment 168004 [details] [diff] [review]
Patch

r=ere
Attachment #168004 - Flags: review?(emaijala) → review+
Attachment #168004 - Flags: superreview?(bryner)
Attachment #168004 - Flags: superreview?(bryner) → superreview?(bzbarsky)
Comment on attachment 168004 [details] [diff] [review]
Patch

sr=bzbarsky
Attachment #168004 - Flags: superreview?(bzbarsky) → superreview+
Ere:

Please check-in the patch.
Comment on attachment 168004 [details] [diff] [review]
Patch

The risk of this patch is low. But this problem is 'major'.
Attachment #168004 - Flags: approval1.8a6?
Comment on attachment 168004 [details] [diff] [review]
Patch

a=asa (on behalf of drivers) for checkin to Mozilla 1.8a6.
Attachment #168004 - Flags: approval1.8a6? → approval1.8a6+
Jungshik:

Could you check-in the patch?
fix checked in.

Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: