Closed
Bug 601487
Opened 14 years ago
Closed 14 years ago
font sanitizer breaks Variation Selector support for downloadable fonts (reftest layout/reftests/font-face/ivs-1.html fails)
Categories
(Core :: Layout: Text and Fonts, defect)
Tracking
()
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
blocking2.0 | --- | final+ |
People
(Reporter: jfkthame, Assigned: jfkthame)
References
Details
Attachments
(1 file)
(deleted),
patch
|
jtd
:
review+
|
Details | Diff | Splinter Review |
This reftest fails because the OTS sanitizer transcodes the 'cmap' table of the font, and in the process it includes only selected subtables. It does not support the format-14 subtable used for Unicode Variation Sequences, and therefore the IVS reftest breaks.
Note that this is a separate issue from bug 600821, which relates to the IVS test font being rejected by the sanitizer due to lack of padding. Once the font is fixed, the sanitizer accepts it, but breaks the reftest because it drops the format 14 subtable and so the variation sequence is not handled properly.
Assignee | ||
Comment 1•14 years ago
|
||
This adds support for the Variation Sequence cmap subtable format to the OTS cmap parser. (The code is based on our gfxFontUtils::ReadCMAPTableFormat14, adapted to the OTS environment and coding style and with additional validation of the glyph IDs.)
Applies on top of the patches from bug 527276, obviously; we'll want to land them all as a group.
Assignee: nobody → jfkthame
Attachment #480513 -
Flags: review?(jdaggett)
Assignee | ||
Comment 2•14 years ago
|
||
Filed upstream as http://code.google.com/p/chromium/issues/detail?id=57766.
Updated•14 years ago
|
Attachment #480513 -
Flags: review?(jdaggett) → review+
Assignee | ||
Comment 3•14 years ago
|
||
Requesting blocking-2.0, as without this patch we'll cause a regression when the font sanitizer (bug 527276) lands - it will disable Variation Selector support (see bug 552460) for downloadable fonts.
blocking2.0: --- → ?
Summary: reftest layout/reftests/font-face/ivs-1.html fails when OTS sanitizer is enabled → font sanitizer breaks Variation Selector support for downloadable fonts (reftest layout/reftests/font-face/ivs-1.html fails)
Updated•14 years ago
|
blocking2.0: ? → final+
Assignee | ||
Comment 4•14 years ago
|
||
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•