Closed Bug 380491 Opened 17 years ago Closed 17 years ago

Fx randomly fails to display non-BMP ideographs

Categories

(Core Graveyard :: GFX: Win32, defect)

Product:
Core Graveyard
Component:
GFX: Win32
Version:
1.8 Branch
Platform:
x86
Windows Vista
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
VERIFIED FIXED

People

(Reporter: emk, Assigned: emk)

Details

(Keywords: intl, jp-critical, verified1.8.1.5, Whiteboard: branch-only)

Attachments

(2 files)

testcase
208.64 KB, text/html; charset=UTF-8
Details
patch (branch only)
1.00 KB, patch
rbs
: review+
rbs
: superreview+
dveditz
: approval1.8.1.5+
Details | Diff | Splinter Review 
Steps to reproduce:
1. Set "MS PGothic", "MS PMincho" or "Meiryo" as default on your font preference.
2. Open the attached testcase.
Actual result:
Many characters are displayed as middle-dot or blank-box.
Expected result:
All characters should be displayed.

This is significant for Japanese users because Vista default Japanes fonts have only a few glyphs in Ext.B range.
Attached file testcase — 

    
    

    
  


  
|new PRUint32[]| DOESN'T initialize the array.
Assignee: win32 → VYV03354
Status: NEW → ASSIGNED

        Attachment #264591 -
        Flags: superreview?(rbs)

        Attachment #264591 -
        Flags: review?(rbs)

    
  

        Attachment #264591 -
        Attachment description: patch → patch (branch only)

    
    

    
  
Comment on attachment 264591 [details] [diff] [review]
patch (branch only)

r+sr=rbs

        Attachment #264591 -
        Flags: superreview?(rbs)

        Attachment #264591 -
        Flags: superreview+

        Attachment #264591 -
        Flags: review?(rbs)

        Attachment #264591 -
        Flags: review+

    
    

    
  
Comment on attachment 264591 [details] [diff] [review]
patch (branch only)

We couldn't test this on trunk because trunk can no longer build without cairo.
This is a simple one-liner and fix a significant problem for Japanese users.

        Attachment #264591 -
        Flags: approval1.8.1.5?

    
  
Flags: blocking1.8.1.5?

    
    

    
  
Comment on attachment 264591 [details] [diff] [review]
patch (branch only)

approved for 1.8.1.5, a=dveditz for release-drivers

But not blocking the release.

        Attachment #264591 -
        Flags: approval1.8.1.5? → approval1.8.1.5+
Flags: blocking1.8.1.5?
Whiteboard: branch-only

    
    

    
  
checked-in.
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Keywords: fixed1.8.1.5
Resolution: --- → FIXED

    
    

    
  
-> v.
using latest nightly Fx2 branch.
Status: RESOLVED → VERIFIED
Keywords: fixed1.8.1.5 → verified1.8.1.5
Product: Core → Core Graveyard

          You need to log in
          before you can comment on or make changes to this bug.
        






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: