aboutsummaryrefslogtreecommitdiffhomepage
path: root/src/core/SkPaint.cpp
diff options
context:
space:
mode:
authorGravatar reed@google.com <reed@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81>2011-09-13 13:23:26 +0000
committerGravatar reed@google.com <reed@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81>2011-09-13 13:23:26 +0000
commit4f79b9be9ea5e8998b3692c1030ae2c90a796e9d (patch)
tree1431515548b8e7b91ef0e589956eddc5b7874fdd /src/core/SkPaint.cpp
parent4c888aae25c1f6a420caceec7c42736ba6ddad3d (diff)
set max size for lcdtext
git-svn-id: http://skia.googlecode.com/svn/trunk@2252 2bbb7eff-a529-9590-31e7-b0007b416f81
Diffstat (limited to 'src/core/SkPaint.cpp')
-rw-r--r--src/core/SkPaint.cpp13
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 {