diff options
| author |  Ben Wagner <bungeman@google.com> | 2017-08-08 15:37:09 -0400 |
|---|---|---|
| committer |  Skia Commit-Bot <skia-commit-bot@chromium.org> | 2017-08-11 14:21:27 +0000 |
| commit | 880768032d7bd1528dc84f5b4408d4fa418bae7d (patch) | |
| tree | 1a53956446f30edaf65b597c7bc78ead537a2274 /src/ports/SkFontHost_FreeType.cpp | |
| parent | 26080be49fdd762976e6864036c2e53d1ac8f452 (diff) | |
Remove subpixel positioning implies no bytecode hinting.
SkTypeface_FreeType::onFilterRec currently assumes that if we're asked
to do subpixel positioning, don't do bytecode hinting. The idea was that
both could not be satisfied at the same time, so pick something. This is
no longer true, as with the v40 interpreter it is possible to get
subpixel positioned but bytcode hinted glyphs.
BUG=skia:6931
Change-Id: Ifaeff20c121d6bb4b9287f552e383547eb6d5d49
Reviewed-on: https://skia-review.googlesource.com/32201
Reviewed-by: Yuqian Li <liyuqian@google.com>
Reviewed-by: Ben Wagner <bungeman@google.com>
Commit-Queue: Ben Wagner <bungeman@google.com>
Diffstat (limited to 'src/ports/SkFontHost_FreeType.cpp')
| -rw-r--r-- | src/ports/SkFontHost_FreeType.cpp | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/src/ports/SkFontHost_FreeType.cpp b/src/ports/SkFontHost_FreeType.cpp index 5af6190d91..ca97371837 100644 --- a/src/ports/SkFontHost_FreeType.cpp +++ b/src/ports/SkFontHost_FreeType.cpp @@ -677,11 +677,6 @@ void SkTypeface_FreeType::onFilterRec(SkScalerContextRec* rec) const { // collapse full->normal hinting if we're not doing LCD h = SkPaint::kNormal_Hinting; } - if ((rec->fFlags & SkScalerContext::kSubpixelPositioning_Flag)) { - if (SkPaint::kNo_Hinting != h) { - h = SkPaint::kSlight_Hinting; - } - } // rotated text looks bad with hinting, so we disable it as needed if (!isAxisAligned(*rec)) { |