Closed
Bug 272199
Opened 21 years ago
Closed 21 years ago
Cannot reconvert with Japanist(Japanese IME) on WinNT
Categories
(Core :: Internationalization, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: masayuki, Assigned: masayuki)
Details
(Keywords: inputmethod, intl)
Attachments
(1 file, 1 obsolete file)
2.47 KB,
patch
|
emaijala+moz
:
review+
bzbarsky
:
superreview+
asa
:
approval1.8a6+
|
Details | Diff | Splinter Review |
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.
Assignee | ||
Comment 1•21 years ago
|
||
I tested with MS-IME2000/ATOK17/Japanist2003 on Win2000/WinME.
Assignee | ||
Updated•21 years ago
|
Attachment #167323 -
Flags: review?(emaijala)
Assignee | ||
Updated•21 years ago
|
Status: NEW → ASSIGNED
Comment 2•21 years ago
|
||
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-
Assignee | ||
Comment 3•21 years ago
|
||
Attachment #167323 -
Attachment is obsolete: true
Assignee | ||
Updated•21 years ago
|
Attachment #168004 -
Flags: review?(emaijala)
Comment 4•21 years ago
|
||
Comment on attachment 168004 [details] [diff] [review]
Patch
r=ere
Attachment #168004 -
Flags: review?(emaijala) → review+
Assignee | ||
Updated•21 years ago
|
Attachment #168004 -
Flags: superreview?(bryner)
Assignee | ||
Updated•21 years ago
|
Attachment #168004 -
Flags: superreview?(bryner) → superreview?(bzbarsky)
![]() |
||
Comment 5•21 years ago
|
||
Comment on attachment 168004 [details] [diff] [review]
Patch
sr=bzbarsky
Attachment #168004 -
Flags: superreview?(bzbarsky) → superreview+
Assignee | ||
Comment 6•21 years ago
|
||
Ere:
Please check-in the patch.
Assignee | ||
Comment 7•21 years ago
|
||
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 8•21 years ago
|
||
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+
Assignee | ||
Comment 9•21 years ago
|
||
Jungshik:
Could you check-in the patch?
Comment 10•21 years ago
|
||
fix checked in.
Status: ASSIGNED → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•15 years ago
|
Keywords: inputmethod
You need to log in
before you can comment on or make changes to this bug.
Description
•