RESOLVED FIXED 117726
Synthesized vertical italics on rotated glyphs are transformed incorrectly
https://bugs.webkit.org/show_bug.cgi?id=117726
Summary Synthesized vertical italics on rotated glyphs are transformed incorrectly
Dean Jackson
Reported 2013-06-17 22:38:07 PDT
Created attachment 204882 [details] Test Case See the testcase. If a document is using vertical layout, and italics, and using a font that does not provide italic glyphs, then the synthesis of italics skews in the wrong direction. <rdar://problem/12783606>
Attachments
Test Case (321 bytes, text/html)
2013-06-17 22:38 PDT, Dean Jackson
no flags
Patch (12.98 KB, patch)
2013-06-17 22:49 PDT, Dean Jackson
no flags
Patch (33.29 KB, patch)
2013-06-18 12:40 PDT, Dean Jackson
no flags
Dean Jackson
Comment 1 2013-06-17 22:49:31 PDT
Early Warning System Bot
Comment 2 2013-06-17 22:57:36 PDT
Early Warning System Bot
Comment 3 2013-06-17 22:59:23 PDT
Eric Carlson
Comment 4 2013-06-18 07:52:42 PDT
Comment on attachment 204883 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=204883&action=review > Source/WebCore/platform/graphics/FontGlyphs.cpp:207 > + RefPtr<SimpleFontData> nonItalicFontData = data.fontData->nonSyntheticItalicFontData(); > + GlyphPageTreeNode* nonItalicNode = GlyphPageTreeNode::getRootChild(nonItalicFontData.get(), pageNumber); > + GlyphPage* nonItalicPage = nonItalicNode->page(); Nit: these aren't used if page is NULL.
Dean Jackson
Comment 5 2013-06-18 10:05:33 PDT
(In reply to comment #4) > (From update of attachment 204883 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=204883&action=review > > > Source/WebCore/platform/graphics/FontGlyphs.cpp:207 > > + RefPtr<SimpleFontData> nonItalicFontData = data.fontData->nonSyntheticItalicFontData(); > > + GlyphPageTreeNode* nonItalicNode = GlyphPageTreeNode::getRootChild(nonItalicFontData.get(), pageNumber); > > + GlyphPage* nonItalicPage = nonItalicNode->page(); > > Nit: these aren't used if page is NULL. That was a typo. It should have been if (nonItalicPage). Thanks for noticing!
Enrica Casucci
Comment 6 2013-06-18 10:53:30 PDT
Comment on attachment 204883 [details] Patch Looks good to me, with the change Eric pointed out. I trust you to fix the 2 qt builds before landing the patch.
Dean Jackson
Comment 7 2013-06-18 12:40:25 PDT
Dean Jackson
Comment 8 2013-06-18 13:30:33 PDT
Note You need to log in before you can comment on or make changes to this bug.