[Webkit-unassigned] [Bug 45318] Complex Path FontCache does not handle SmallCaps properly

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=45318


Gary Simmons <gsimmons at rim.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #66754|0                           |1
        is obsolete|                            |
  Attachment #66916|                            |review?
               Flag|                            |




--- Comment #5 from Gary Simmons <gsimmons at rim.com>  2010-09-08 11:07:19 PST ---
Created an attachment (id=66916)
 --> (https://bugs.webkit.org/attachment.cgi?id=66916)
Updated patch with LayoutTests

I've updated the patch and attached a relevant layout test. This bug only manifests itself in the Qt port as all other ports are now using the fast path for fonts. When using the non-fast path code, Small Caps is requested as a variant attribute in the FontDescription. With the current code, the generated FontPlatformData object will be indistinguishable (in terms of hash keys) from the plain variant causing whichever one is used first to become always used.

If the test is manually loaded in the current Qt browser, you'll see all the text as small caps (which is incorrect as described by the test and demonstrated by other browsers).

Any ports using the fast path are exempt from this issue as they request the smallCapsFontData from the SimpleFontData object rather than directly asking the cache. 

I have generated expected results for Qt but I have not had access to the resources to generate results for other ports as of yet.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.