RESOLVED FIXED Bug 144305
Implement font-synthesis CSS property 
https://bugs.webkit.org/show_bug.cgi?id=144305
Summary Implement font-synthesis CSS property
Myles C. Maxfield
Reported 2015-04-27 19:16:54 PDT
Implement font-synthesis CSS property
Attachments
Patch (28.44 KB, patch)
2015-04-27 20:57 PDT, Myles C. Maxfield 		no flags
DetailsFormatted DiffDiff
Patch (33.35 KB, patch)
2015-04-27 21:08 PDT, Myles C. Maxfield 		no flags
DetailsFormatted DiffDiff
Patch (33.35 KB, patch)
2015-04-28 11:02 PDT, Myles C. Maxfield 		kling: review+
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Myles C. Maxfield
Comment 1 2015-04-27 20:57:22 PDT
Created attachment 251818 [details] Patch
Myles C. Maxfield
Comment 2 2015-04-27 20:58:20 PDT
<rdar://problem/20692791>
Myles C. Maxfield
Comment 3 2015-04-27 21:08:55 PDT
Created attachment 251819 [details] Patch
Jon Lee
Comment 4 2015-04-27 23:37:30 PDT
Comment on attachment 251819 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=251819&action=review > Source/WebCore/platform/graphics/ios/FontCacheIOS.mm:698 > + bool syntheticOblique = (fontDescription.fontSynthesis & FontSynthesisStyle) && (traits & kCTFontTraitItalic) && !(actualTraits & kCTFontTraitItalic) && !isAppleColorEmoji; Missing ()
Myles C. Maxfield
Comment 5 2015-04-28 11:02:35 PDT
Created attachment 251863 [details] Patch
Andreas Kling
Comment 6 2015-04-28 11:25:15 PDT
Comment on attachment 251863 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=251863&action=review r=me, looks good. > Source/WebCore/css/StyleBuilderConverter.h:1224 > + for (CSSValue& i : downcast<CSSValueList>(value)) { 'i' is kind of a weird name for a CSSValue& :)
Myles C. Maxfield
Comment 7 2015-04-28 11:38:14 PDT
Committed r183494: <http://trac.webkit.org/changeset/183494>
Note You need to log in before you can comment on or make changes to this bug.