Move distance field font code into SkGpuDevice

src/core/SkDraw.cpp

 /*
  * Copyright 2006 The Android Open Source Project
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #include "SkDraw.h"
 #include "SkBlitter.h"
 #include "SkBounder.h"
@@ skipping 14 matching lines @@
 #include "SkStroke.h"
 #include "SkTemplatesPriv.h"
 #include "SkTLazy.h"
 #include "SkUtils.h"
 
 #include "SkAutoKern.h"
 #include "SkBitmapProcShader.h"
 #include "SkDrawProcs.h"
 #include "SkMatrixUtils.h"
 
-bool SkDraw::ShouldDrawTextAsPaths(const SkPaint& paint, const SkMatrix& ctm) {
-    // we don't cache hairlines in the cache
-    if (SkPaint::kStroke_Style == paint.getStyle() &&
-            0 == paint.getStrokeWidth()) {
-        return true;
-    }
-
-    // we don't cache perspective
-    if (ctm.hasPerspective()) {
-        return true;
-    }
-
-    SkMatrix textM;
-    return SkPaint::TooBigToUseCache(ctm, *paint.setTextMatrix(&textM));
-}
-
 //#define TRACE_BITMAP_DRAWS
 
 #define kBlitterStorageLongCount    (sizeof(SkBitmapProcShader) >> 2)
 
 /** Helper for allocating small blitters on the stack.
  */
 class SkAutoBlitterChoose : SkNoncopyable {
 public:
     SkAutoBlitterChoose() {
         fBlitter = NULL;
@@ skipping 1390 matching lines @@
         const SkGlyph& glyph = glyphCacheProc(cache, &text, 0, 0);
 
         x += autokern.adjust(glyph) + glyph.fAdvanceX;
         y += glyph.fAdvanceY;
     }
     stopVector->set(SkFixedToScalar(x), SkFixedToScalar(y));
 
     SkASSERT(text == stop);
 }
 
+bool SkDraw::shouldDrawTextAsPaths(const SkPaint& paint, const SkMatrix& ctm) const {
+    // we don't cache hairlines in the cache

[robertphillips, 2013/11/25 20:39:44]

line up? or 1 line?
Shouldn't this comment be "// drawing hairlines is fast so it doesn't make any
sense to cache them"?

+    if (SkPaint::kStroke_Style == paint.getStyle() &&
+            0 == paint.getStrokeWidth()) {
+        return true;
+    }
+
+    // we don't cache perspective
+    if (ctm.hasPerspective()) {
+        return true;
+    }
+
+    SkMatrix textM;
+    return SkPaint::TooBigToUseCache(ctm, *paint.setTextMatrix(&textM));
+}
+
 void SkDraw::drawText_asPaths(const char text[], size_t byteLength,
                               SkScalar x, SkScalar y,
                               const SkPaint& paint) const {
     SkDEBUGCODE(this->validate();)
 
     SkTextToPathIter iter(text, byteLength, paint, true);
 
     SkMatrix    matrix;
     matrix.setScale(iter.getPathScale(), iter.getPathScale());
     matrix.postTranslate(x, y);
@@ skipping 230 matching lines @@
     SkASSERT(byteLength == 0 || text != NULL);
 
     SkDEBUGCODE(this->validate();)
 
     // nothing to draw
     if (text == NULL || byteLength == 0 || fRC->isEmpty()) {
         return;
     }
 
     // SkScalarRec doesn't currently have a way of representing hairline stroke and
     // will fill if its frame-width is 0.

[robertphillips, 2013/11/25 20:39:44]

this->

-    if (ShouldDrawTextAsPaths(paint, *fMatrix)) {
+    if (shouldDrawTextAsPaths(paint, *fMatrix)) {
         this->drawText_asPaths(text, byteLength, x, y, paint);
         return;
     }
 
     SkDrawCacheProc glyphCacheProc = paint.getDrawCacheProc();
 
-#if SK_DISTANCEFIELD_FONTS
-    const SkMatrix* ctm = fMatrix;
-    const SkPaint* paintRef = &paint;
-    SkPaint paintCopy;
-    uint32_t procFlags = fProcs ? fProcs->fFlags : 0;
-    if (procFlags & SkDrawProcs::kUseScaledGlyphs_Flag) {
-        paintCopy = paint;
-        paintCopy.setTextSize(SkDrawProcs::kBaseDFFontSize);
-        paintCopy.setLCDRenderText(false);
-        paintRef = &paintCopy;
-    }
-    if (procFlags & SkDrawProcs::kSkipBakedGlyphTransform_Flag) {
-        ctm = NULL;
-    }
-    SkAutoGlyphCache    autoCache(*paintRef, &fDevice->fLeakyProperties, ctm);
-#else
     SkAutoGlyphCache    autoCache(paint, &fDevice->fLeakyProperties, fMatrix);
-#endif
     SkGlyphCache*       cache = autoCache.getCache();
 
     // transform our starting point
-#if SK_DISTANCEFIELD_FONTS
-    if (!(procFlags & SkDrawProcs::kSkipBakedGlyphTransform_Flag))
-#endif
     {
         SkPoint loc;
         fMatrix->mapXY(x, y, &loc);
         x = loc.fX;
         y = loc.fY;
     }
 
     // need to measure first
     if (paint.getTextAlign() != SkPaint::kLeft_Align) {
         SkVector    stop;
@@ skipping 38 matching lines @@
             d1g.fHalfSampleY = SK_FixedHalf;
         } else if (kY_SkAxisAlignment == baseline) {
             fxMask = 0;
             d1g.fHalfSampleX = SK_FixedHalf;
         }
     }
 
     SkFixed fx = SkScalarToFixed(x) + d1g.fHalfSampleX;
     SkFixed fy = SkScalarToFixed(y) + d1g.fHalfSampleY;
 
-#if SK_DISTANCEFIELD_FONTS
-    SkFixed fixedScale;
-    if (procFlags & SkDrawProcs::kUseScaledGlyphs_Flag) {
-        fixedScale = SkScalarToFixed(paint.getTextSize()/(float)SkDrawProcs::kBaseDFFontSize);
-    }
-#endif
     while (text < stop) {
         const SkGlyph& glyph = glyphCacheProc(cache, &text, fx & fxMask, fy & fyMask);
 
-#if SK_DISTANCEFIELD_FONTS
-        if (procFlags & SkDrawProcs::kUseScaledGlyphs_Flag) {
-            fx += SkFixedMul_portable(autokern.adjust(glyph), fixedScale);
-        } else {
-            fx += autokern.adjust(glyph);
-        }
-#else
         fx += autokern.adjust(glyph);
-#endif
 
         if (glyph.fWidth) {
             proc(d1g, fx, fy, glyph);
         }
 
-#if SK_DISTANCEFIELD_FONTS
-        if (procFlags & SkDrawProcs::kUseScaledGlyphs_Flag) {
-            fx += SkFixedMul_portable(glyph.fAdvanceX, fixedScale);
-            fy += SkFixedMul_portable(glyph.fAdvanceY, fixedScale);
-        } else {
-            fx += glyph.fAdvanceX;
-            fy += glyph.fAdvanceY;
-        }
-#else
         fx += glyph.fAdvanceX;
         fy += glyph.fAdvanceY;
-#endif
     }
 }
 
 // last parameter is interpreted as SkFixed [x, y]
 // return the fixed position, which may be rounded or not by the caller
 //   e.g. subpixel doesn't round
 typedef void (*AlignProc)(const SkPoint&, const SkGlyph&, SkIPoint*);
 
 static void leftAlignProc(const SkPoint& loc, const SkGlyph& glyph,
                           SkIPoint* dst) {
@@ skipping 129 matching lines @@
                          int scalarsPerPosition, const SkPaint& paint) const {
     SkASSERT(byteLength == 0 || text != NULL);
     SkASSERT(1 == scalarsPerPosition || 2 == scalarsPerPosition);
 
     SkDEBUGCODE(this->validate();)
 
     // nothing to draw
     if (text == NULL || byteLength == 0 || fRC->isEmpty()) {
         return;
     }
 

[robertphillips, 2013/11/25 20:39:44]

this->

-    if (ShouldDrawTextAsPaths(paint, *fMatrix)) {
+    if (shouldDrawTextAsPaths(paint, *fMatrix)) {
         this->drawPosText_asPaths(text, byteLength, pos, constY,
                                   scalarsPerPosition, paint);
         return;
     }
 
     SkDrawCacheProc     glyphCacheProc = paint.getDrawCacheProc();
-#if SK_DISTANCEFIELD_FONTS
-    const SkMatrix* ctm = fMatrix;
-    const SkPaint* paintRef = &paint;
-    SkPaint paintCopy;
-    uint32_t procFlags = fProcs ? fProcs->fFlags : 0;
-    if (procFlags & SkDrawProcs::kUseScaledGlyphs_Flag) {
-        paintCopy = paint;
-        paintCopy.setTextSize(SkDrawProcs::kBaseDFFontSize);
-        paintRef = &paintCopy;
-    }
-    if (procFlags & SkDrawProcs::kSkipBakedGlyphTransform_Flag) {
-        ctm = &SkMatrix::I();
-    }
-    SkAutoGlyphCache    autoCache(*paintRef, &fDevice->fLeakyProperties, ctm);
-#else
     SkAutoGlyphCache    autoCache(paint, &fDevice->fLeakyProperties, fMatrix);
-#endif
     SkGlyphCache*       cache = autoCache.getCache();
 
     SkAAClipBlitterWrapper wrapper;
     SkAutoBlitterChoose blitterChooser;
     SkBlitter* blitter = NULL;
     if (needsRasterTextBlit(*this)) {
         blitterChooser.choose(*fBitmap, *fMatrix, paint);
         blitter = blitterChooser.get();
         if (fRC->isAA()) {
             wrapper.init(*fRC, blitter);
             blitter = wrapper.getBlitter();
         }
     }
 
     const char*        stop = text + byteLength;
     AlignProc          alignProc = pick_align_proc(paint.getTextAlign());
     SkDraw1Glyph       d1g;
     SkDraw1Glyph::Proc proc = d1g.init(this, blitter, cache, paint);
-#if SK_DISTANCEFIELD_FONTS
-    TextMapState       tms(*ctm, constY);
-#else
     TextMapState       tms(*fMatrix, constY);
-#endif
     TextMapState::Proc tmsProc = tms.pickProc(scalarsPerPosition);
 
     if (cache->isSubpixel()) {
         // maybe we should skip the rounding if linearText is set
         SkAxisAlignment baseline = SkComputeAxisAlignmentForHText(*fMatrix);
 
         SkFixed fxMask = ~0;
         SkFixed fyMask = ~0;
         if (kX_SkAxisAlignment == baseline) {
             fyMask = 0;
 #ifndef SK_IGNORE_SUBPIXEL_AXIS_ALIGN_FIX
             d1g.fHalfSampleY = SK_FixedHalf;
 #endif
         } else if (kY_SkAxisAlignment == baseline) {
             fxMask = 0;
 #ifndef SK_IGNORE_SUBPIXEL_AXIS_ALIGN_FIX
             d1g.fHalfSampleX = SK_FixedHalf;
 #endif
         }
 
         if (SkPaint::kLeft_Align == paint.getTextAlign()) {
             while (text < stop) {
-#if SK_DISTANCEFIELD_FONTS
-                if (procFlags & SkDrawProcs::kSkipBakedGlyphTransform_Flag) {
-                    tms.fLoc.fX = *pos;
-                    tms.fLoc.fY = *(pos+1);
-                } else {
-                    tmsProc(tms, pos);
-                }
-#else
                 tmsProc(tms, pos);
-#endif
                 SkFixed fx = SkScalarToFixed(tms.fLoc.fX) + d1g.fHalfSampleX;
                 SkFixed fy = SkScalarToFixed(tms.fLoc.fY) + d1g.fHalfSampleY;
 
                 const SkGlyph& glyph = glyphCacheProc(cache, &text,
                                                       fx & fxMask, fy & fyMask);
 
                 if (glyph.fWidth) {
                     proc(d1g, fx, fy, glyph);
                 }
                 pos += scalarsPerPosition;
@@ skipping 797 matching lines @@
         mask->fImage = SkMask::AllocImage(size);
         memset(mask->fImage, 0, mask->computeImageSize());
     }
 
     if (SkMask::kJustComputeBounds_CreateMode != mode) {
         draw_into_mask(*mask, devPath, style);
     }
 
     return true;
 }