REGRESSION(Variation Fonts): Specifying Skia by PostScript name does not yield the expected result
Created attachment 314423 [details] Patch
Created attachment 314424 [details] Patch
Created attachment 314427 [details] Patch
<rdar://problem/33040854>
Comment on attachment 314427 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=314427&action=review > Source/WebCore/platform/graphics/cocoa/FontCacheCoreText.cpp:1165 > + return { adoptCF(CTFontCreateWithFontDescriptor(installedFont->fontDescriptor.get(), size, nullptr)), false }; This false is unnecessary. Does it help with readability?
Committed r219102: <http://trac.webkit.org/changeset/219102>
Comment on attachment 314427 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=314427&action=review >> Source/WebCore/platform/graphics/cocoa/FontCacheCoreText.cpp:1165 >> + return { adoptCF(CTFontCreateWithFontDescriptor(installedFont->fontDescriptor.get(), size, nullptr)), false }; > > This false is unnecessary. Does it help with readability? Yeah. In the other cases it doesn't matter, but in this case, it's explicitly necessary.