49559 - JPN characters disappear at the beginning of a line when they are an anchor for an URL link

Status: VERIFIED FIXED

(Core :: Internationalization, defect, P1)

Description

[Katsuhiko Momoi]

** Observed with 8/18/2000 Win32 M18 build **
** Observed (so far) only under Win 98-Japanese **
** This bug was first reported by a JPN Mozilla volunteer. **

Problem:

1. Create a HREF link and as part of the anchor, insert some Japanese characters. For example,

   <A HREF=http://.......> JPN_word</A>

2. When displaying this line, make the Browser window smaller so that the right edge of the window
   is slightly in front of the anchor word created above.
3. So at this point, all characters of "JPN_word" will have moved to 2nd line.
4. But you see that in this particular case, a couple of JPN characters have disppeared.
5. Now if you reload from the source again, the 2 characters get displayed.
   Looks like somehow re-painting does not occur completely for this type of 
   words when the entire word moves to the next line.

Additional notes:

I tried the test case below under Japanese 98 and NT4 using the same 8/18/2000 M18
build. But so far I have been able to reproduce this problem under only Win 98-Japanese.


I'll attach 2 images below illustrating what the problem looks like.
Also the test case.

Comment 1

[Katsuhiko Momoi]

Attachment: This is how the line should break on an anchored word. See the next image for missing chars.

Comment 2

[Katsuhiko Momoi]

Look at the 2nd line of this image.
Now contrast this with the 2nd image when the line break just occurred.

Comment 3

[Katsuhiko Momoi]

Attachment: This shows the condition right after the 1st line was broken in front of the anchor word to an URL.

Comment 4

[Katsuhiko Momoi]

I'll attach below the test case provided by a JPN Mozilla volunteer.

Comment 5

[Katsuhiko Momoi]

Attachment: The test case in HTML format which is shown in 2 images above.

Comment 6

[Katsuhiko Momoi]

Note that when you make the Brower window smaller to the point when
the anchor word moves to the next line, the word

メーリングリスト

shows as 

リングリスト

As I said above, I tried this a number of times on NT4 but could not reproduce it.
On Win 98-J, it is very to reproduce this problem.

Comment 7

[Frank Tang]

The test case (attachment 3 [details] [diff] [review]) is in EUC-JP.
reassign this to shanjian and mark it nsbeta3
It looks the line break correctly. You can see the first line have the space for 
the first two characters in the japanese link. However, some code does not paint 
it when it should. 
Have we test Linux/Mac ?
Have we test Win95J/NTJ ? Win95US / NTUS ?

Comment 8

[Frank Tang]

IF this problem also happen on Linux/Mac and all other Window. Then this is a 
generic problem
If this problem only happen on Window. Then this may related to the MeasureText 
routine that troy #ifdef XP_WIN (or XP_PC) 
If this problem only happen on Japanese version of Win95/98 but not Japanese NT 
nor US 95/NT, then it is related to the GFX code erik work around the Japanese 
95/98 GDI crashing problem

nsbeta3

Comment 9

[Frank Tang]

     IF this problem also happen on Linux/Mac and all other Window. Then this is 
a 
     generic problem
     If this problem only happen on Window. Then this may related to the 
MeasureText 
     routine that troy #ifdef XP_WIN (or XP_PC) 
     If this problem only happen on Japanese version of Win95/98 but not 
Japanese NT 
     nor US 95/NT, then it is related to the GFX code erik work around the 
Japanese 
     95/98 GDI crashing problem

Comment 10

[Frank Tang]

I cannot reproduce this problem on Mac nsbeta2

Comment 11

[Frank Tang]

I think the bug is in those #ifdef _WIN32 code inside the nsTextFrame.cpp . If I 
change them into #ifdef 0 and recompile it, I cannot reproduce them. So, we 
should look at the #ifdef _WIN32 code in the nsTextFrame.cpp

reassign to shanjian

Comment 12

[Jaime Rodriguez, Jr.]

Marking nsbeta3+ per I18N bug triage.

Comment 13

[Shanjian Li]

Attachment: proposed fix

Comment 14

[Frank Tang]

Need some discussion.

Comment 15

[Shanjian Li]

fix checked in.

Comment 16

[Teruko Kobayashi]

I verified this in 2000-09-05-08 Win32 build.