127129 - The above page is displayed many blank areas

Status: VERIFIED FIXED

(Core :: Internationalization, defect)

Description

[Yuying Long]

Go to the page:
http://law.china.com/zh_cn/zt/alpc/10002958/20020116/10192949.html

Many areas are displayed blank in this page.

This is a regression problem, it happened sometime between 02-12 and 02-18 trunk
build.

Comment 1

[Yuying Long]

-> nsbeta1, regression

It happens on both RH7.1JA and 6.2JA.

Comment 2

[Yuying Long]

Attachment: screen shot for part of this page

Comment 3

[Shanjian Li]

The problem does not happen with 2/12 nightly build, but 2/13 build has this
problem. 
reassign to myself. 

frank, we should nominate this one as nsbeta1+.

Comment 4

[Yuying Long]

Attachment: Saved this html page

Comment 5

[Shanjian Li]

Add brian to cc list. 

Comment 6

[Dimitrios]

There is at least one <table> tag with "height" attribute inside. Sothis bug
might be a duplicate of bug 126742. Could someone with required fonts installed
remove the above attribute and then check if the page renders as expected?

Comment 7

[Shanjian Li]

I debugged into the X font code and inside debug everythings runs fine even to
the point to DrawString. Analysis of the checkins between 2/12 and 2/13 also
seems suggesting that it is not a X font problem. NS_FONT_DEBUG does not reveal
anything useful. Since the problem is most likely caused by a patch in other
area, and I do not find any clue about which part might be the cause, it is
reasonable to me to further check and hopefully identify the problematic checkin. 

Comment 8

[Shanjian Li]

This is a regression caused by frank's fix for bug 101998. I saw all the
character has been converted to unicode correctly in my debug session, that made
me belive the patch was irrevant to this problem. I just realized that we need
to convert from unicode to GB2312GL before the glyph can be found, and one file
touched by 101998 changed the behavior. Patch will follow soon. 

Comment 9

[Shanjian Li]

Attachment: patch

Comment 10

[Shanjian Li]

frank, could you review my patch?
When converting from unicode to GB2312GL, we need to clear 8bit. That is skipped
in your patch for 101998.

Comment 11

[Frank Tang]

nsbeta1+ 

Comment 12

[Frank Tang]

Comment on attachment 71551 [details] [diff] [review]
patch 

r=ftang. That make sense

Comment 13

[Shanjian Li]

alec, could you sr? thanks.

Comment 14

[Alec Flett]

Comment on attachment 71551 [details] [diff] [review]
patch 

sr=alecf

Comment 15

[Asa Dotzler [:asa]]

Comment on attachment 71551 [details] [diff] [review]
patch 

a=asa (on behalf of drivers) for checkin to the 1.0 trunk

Comment 16

[Shanjian Li]

fix checked in to trunk. 

Comment 17

[Yuying Long]

Verified fixed on 03-06 trunk build / RH7.2.

Comment 18

[wacheo]

btw, will the patch be checked in to 0.9.9? this bug doesn't have target
milestone set...

Comment 19

[Roy Yokoyama]

setting milestone to 1.0

Comment 20

[Yuying Long]

*** Bug 130479 has been marked as a duplicate of this bug. ***

Comment 21

[Yuying Long]

*** Bug 130001 has been marked as a duplicate of this bug. ***

Comment 22

[Yuying Long]

*** Bug 130173 has been marked as a duplicate of this bug. ***

Comment 23

[Yuying Long]

*** Bug 138335 has been marked as a duplicate of this bug. ***