Change large glyph check back to > threshold.
authorjvanverth <jvanverth@google.com>
Mon, 22 Jun 2015 15:08:09 +0000 (08:08 -0700)
committerCommit bot <commit-bot@chromium.org>
Mon, 22 Jun 2015 15:08:09 +0000 (08:08 -0700)
Using >= misses some Chromium test cases that are still
being rendered as paths.

BUG=chromium:467569

Review URL: https://codereview.chromium.org/1202433002

gm/mixedtextblobs.cpp
src/gpu/GrAtlasTextContext.cpp

index fb1b632fb9f9dc79e708c8d1d2d08ae31ec78941..ab7beb909b5d3a32afd682ccd21177660bb0bf8f 100644 (file)
@@ -54,7 +54,7 @@ protected:
         // make textblob
         // Text so large we draw as paths
         SkPaint paint;
-        paint.setTextSize(384);
+        paint.setTextSize(385);
         const char* text = "O";
         sk_tool_utils::set_portable_typeface(&paint);
 
index aa51cd2c30bd1be2e707d3f558ea40be000fd2af..c25ddd25e2d7774733f4ff1d2cafb4d713abca65 100644 (file)
@@ -442,7 +442,7 @@ inline bool GrAtlasTextContext::canDrawAsDistanceFields(const SkPaint& skPaint,
     SkScalar scaledTextSize = maxScale*skPaint.getTextSize();
     // Hinted text looks far better at small resolutions
     // Scaling up beyond 2x yields undesireable artifacts
-    if (scaledTextSize < kMinDFFontSize || scaledTextSize >= kLargeDFFontLimit) {
+    if (scaledTextSize < kMinDFFontSize || scaledTextSize > kLargeDFFontLimit) {
         return false;
     }