Skip to content

Commit

Permalink
[Skia] Do not set synthetic bold offset
Browse files Browse the repository at this point in the history
https://bugs.webkit.org/show_bug.cgi?id=270291

Reviewed by Don Olmstead.

It's handled internally by Skia.

* Source/WebCore/platform/graphics/skia/FontSkia.cpp:
(WebCore::Font::platformPathForGlyph const):
(WebCore::Font::platformInit):

Canonical link: https://commits.webkit.org/275541@main
  • Loading branch information
carlosgcampos committed Mar 1, 2024
1 parent 6a7b2d6 commit c7ee831
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions Source/WebCore/platform/graphics/skia/FontSkia.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,6 @@ Path Font::platformPathForGlyph(Glyph glyph) const
auto path = PathSkia::create();
const auto& font = m_platformData.skFont();
font.getPath(glyph, path->platformPath());
// FIXME: syntetic bold offset.
return { path };
}

Expand Down Expand Up @@ -109,8 +108,6 @@ void Font::platformInit()

m_fontMetrics.setUnitsPerEm(font.getTypeface()->getUnitsPerEm());

m_syntheticBoldOffset = m_platformData.syntheticBold() ? 1.0f : 0.f;

SkString familyName;
font.getTypeface()->getFamilyName(&familyName);
if (equalIgnoringASCIICase(familyName.c_str(), "Ahem"_s))
Expand Down

0 comments on commit c7ee831

Please sign in to comment.