226340 - Implement mLoadedFonts on OS/2

Status: RESOLVED FIXED

(Core Graveyard :: GFX: OS/2, defect)

Description

[jhp (no longer active)]

We should implement the use of the mLoadedFonts array as used in Win32 (and
other platforms).  Use of this array to hold selected fonts should give us
performance boost and make the code look cleaner.

Comment 1

[jhp (no longer active)]

Attachment: patch (cvs diff -u)

This patch does several things:
1) Implements mLoadedFonts array.  This makes the code look cleaner, and should
increase performance, since we are doing less creation and destruction of
nsFontOS2 objects.
2) Update to the unicode handling, particularly in ResolveForwards/Backwards. 
Makes use of mLoadedFonts array, but also persists the font objects through the
loop.
3) Added some debugging code, to track font selection, and nsFontOS2 creation
and destruction.
4) White space cleanup in several areas.

Comment 2

[jhp (no longer active)]

Attachment: patch (cvs diff -uw)

Same patch, but ignoring white space changes

Comment 3

[Mike Kaply [:mkaply]]

Comment on attachment 135997 [details] [diff] [review]
patch (cvs diff -u)

r=mkaply

Nice work

Comment 4

[Mike Kaply [:mkaply]]

checked in.