Closed Bug 272199 Opened 21 years ago Closed 21 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)
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: 21 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: