diff options
author | reed@google.com <reed@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> | 2011-09-13 13:23:26 +0000 |
---|---|---|
committer | reed@google.com <reed@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> | 2011-09-13 13:23:26 +0000 |
commit | 4f79b9be9ea5e8998b3692c1030ae2c90a796e9d (patch) | |
tree | 1431515548b8e7b91ef0e589956eddc5b7874fdd | |
parent | 4c888aae25c1f6a420caceec7c42736ba6ddad3d (diff) |
set max size for lcdtext
git-svn-id: http://skia.googlecode.com/svn/trunk@2252 2bbb7eff-a529-9590-31e7-b0007b416f81
-rw-r--r-- | src/core/SkPaint.cpp | 13 |
1 files changed, 12 insertions, 1 deletions
diff --git a/src/core/SkPaint.cpp b/src/core/SkPaint.cpp index 6d2e34dd14..0b8ec22f82 100644 --- a/src/core/SkPaint.cpp +++ b/src/core/SkPaint.cpp @@ -1224,6 +1224,17 @@ static SkPaint::Hinting computeHinting(const SkPaint& paint) { return h; } +// Beyond this size, LCD doesn't appreciably improve quality, but it always +// cost more RAM and draws slower, so we set a cap. +static const SkScalar SK_MAX_SIZE_FOR_LCDTEXT = SkIntToScalar(48); + +static bool tooBigForLCD(const SkScalerContext::Rec& rec) { + SkScalar area = SkScalarMul(rec.fPost2x2[0][0], rec.fPost2x2[1][1]) - + SkScalarMul(rec.fPost2x2[1][0], rec.fPost2x2[0][1]); + SkScalar size = SkScalarMul(area, rec.fTextSize); + return SkScalarAbs(size) > SK_MAX_SIZE_FOR_LCDTEXT; +} + /* * Return the scalar with only limited fractional precision. Used to consolidate matrices * that vary only slightly when we create our key into the font cache, since the font scaler @@ -1308,7 +1319,7 @@ void SkScalerContext::MakeRec(const SkPaint& paint, { SkFontHost::LCDOrder order = SkFontHost::GetSubpixelOrder(); SkFontHost::LCDOrientation orient = SkFontHost::GetSubpixelOrientation(); - if (SkFontHost::kNONE_LCDOrder == order) { + if (SkFontHost::kNONE_LCDOrder == order || tooBigForLCD(*rec)) { // eeek, can't support LCD rec->fMaskFormat = SkMask::kA8_Format; } else { |