90910 - Long word is truncated on conversion

Status: VERIFIED FIXED

(Core :: Internationalization, defect, P2)

Description

[Koike Kazuhiko]

Japanese string which has over 31 characters is truncated on IME
conversion.

(1) Register a long word. For example,

Yomi: "Bagujira"
Goku: "http://www.mozilla.gr.jp/docs/beginbugzilla.html"

(2) Enter "Bagujira" and hit space bar to convert it. The result is:

"http://www.mozilla.gr.jp/docs/b"

Build: 2001071408/Win2k

Comment 1

[nhottanscp]

Reassign to yokoyama, cc to shanjian.

Comment 2

[Teruko Kobayashi]

Changed QA contact to [e-mail].

Comment 3

[Roy Yokoyama]

accepting and setting the milestone to 0.9.3 to see if I can fix this in time.

Comment 4

[Roy Yokoyama]

Moving milestone to 0.9.4 

Comment 5

[Frank Tang]

I guess it could be problem in the following lines in
mozilla/widget/src/windows/nsWindow.cpp

5046:        mIMEAttributeString = new PRUint8[attrStrLen+32];
5047:        mIMEAttributeStringSize = attrStrLen+32;
5056:        mIMEAttributeString = new PRUint8[attrStrLen+32];
5057:        mIMEAttributeStringSize = attrStrLen+32;
5079:        mIMECompClauseString = new PRUint32 [compClauseLen+32];
5080:        mIMECompClauseStringSize = compClauseLen+32;
5096:        mIMECompClauseString = new PRUint32 [compClauseLen+32];
5097:        mIMECompClauseStringSize = compClauseLen+32;

or could around it. 

move to m0.9.5

Comment 6

[msanz]

nsbranch- since Frank moved it to 0.9.5

Comment 7

[Roy Yokoyama]

Attachment: Need to specify mIMECompUnicode->mCapacity in byte

Comment 8

[Roy Yokoyama]

shanjian: can you review my patch?

Comment 9

[Shanjian Li]

That's a good catch. r=shanjian

Comment 10

[Roy Yokoyama]

waterson: can you /sr=?

Comment 11

[Chris Waterson]

Comment on attachment 50189 [details] [diff] [review]
Need to specify mIMECompUnicode->mCapacity in byte

sr=waterson

Comment 12

[Roy Yokoyama]

checked into the trunk.

Comment 13

[Yuying Long]

Fixed verified on 10-01 trunk build / Win2k.