Revert of Modifying TextBlobCacheTest to use SkRandomScalerContext

src/fonts/SkRandomScalerContext.cpp

 /*
  * Copyright 2015 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #include "SkRandomScalerContext.h"
 #include "SkGlyph.h"
 #include "SkPath.h"
 #include "SkCanvas.h"
-#include "SkRasterizer.h"
 
 class SkRandomScalerContext : public SkScalerContext {
 public:
-    SkRandomScalerContext(SkRandomTypeface*, const SkDescriptor*, bool fFakeIt);
+    SkRandomScalerContext(SkRandomTypeface*, const SkDescriptor*);
     virtual ~SkRandomScalerContext();
 
 protected:
     unsigned generateGlyphCount() override;
     uint16_t generateCharToGlyph(SkUnichar) override;
     void generateAdvance(SkGlyph*) override;
     void generateMetrics(SkGlyph*) override;
     void generateImage(const SkGlyph&) override;
     void generatePath(const SkGlyph&, SkPath*) override;
     void generateFontMetrics(SkPaint::FontMetrics*) override;
 
 private:
     SkRandomTypeface*     fFace;
     SkScalerContext* fProxy;
     SkMatrix         fMatrix;
-    bool fFakeIt;
 };
 
 #define STD_SIZE    1
 
 #include "SkDescriptor.h"
 
-SkRandomScalerContext::SkRandomScalerContext(SkRandomTypeface* face, const SkDescriptor* desc,
-                                             bool fakeIt)
+SkRandomScalerContext::SkRandomScalerContext(SkRandomTypeface* face, const SkDescriptor* desc)
         : SkScalerContext(face, desc)
-        , fFace(face)
-        , fFakeIt(fakeIt) {
+        , fFace(face) {
     size_t  descSize = SkDescriptor::ComputeOverhead(1) + sizeof(SkScalerContext::Rec);
     SkAutoDescriptor ad(descSize);
     SkDescriptor*    newDesc = ad.getDesc();
 
     newDesc->init();
     void* entry = newDesc->addEntry(kRec_SkDescriptorTag,
                                     sizeof(SkScalerContext::Rec), &fRec);
     {
         SkScalerContext::Rec* rec = (SkScalerContext::Rec*)entry;
         rec->fTextSize = STD_SIZE;
@@ skipping 27 matching lines @@
     fProxy->getAdvance(glyph);
 
     SkVector advance;
     fMatrix.mapXY(SkFixedToScalar(glyph->fAdvanceX),
                   SkFixedToScalar(glyph->fAdvanceY), &advance);
     glyph->fAdvanceX = SkScalarToFixed(advance.fX);
     glyph->fAdvanceY = SkScalarToFixed(advance.fY);
 }
 
 void SkRandomScalerContext::generateMetrics(SkGlyph* glyph) {
+    fProxy->getAdvance(glyph);
+
+    SkVector advance;
+    fMatrix.mapXY(SkFixedToScalar(glyph->fAdvanceX),
+                  SkFixedToScalar(glyph->fAdvanceY), &advance);
+    glyph->fAdvanceX = SkScalarToFixed(advance.fX);
+    glyph->fAdvanceY = SkScalarToFixed(advance.fY);
+
+    SkPath path;
+    fProxy->getPath(*glyph, &path);
+    path.transform(fMatrix);
+
+    SkRect storage;
+    const SkPaint& paint = fFace->paint();
+    const SkRect& newBounds = paint.doComputeFastBounds(path.getBounds(),
+                                                        &storage,
+                                                        SkPaint::kFill_Style);
+    SkIRect ibounds;
+    newBounds.roundOut(&ibounds);
+    glyph->fLeft = ibounds.fLeft;
+    glyph->fTop = ibounds.fTop;
+    glyph->fWidth = ibounds.width();
+    glyph->fHeight = ibounds.height();
+
     // Here we will change the mask format of the glyph
     // NOTE this is being overridden by the base class
     SkMask::Format format;
-    switch (glyph->getGlyphID() % 3) {
+    switch (glyph->getGlyphID() % 6) {
         case 0:
             format = SkMask::kLCD16_Format;
             break;
         case 1:
             format = SkMask::kA8_Format;
             break;
         case 2:
             format = SkMask::kARGB32_Format;
             break;
+        default:
+            // we will fiddle with these in generate image
+            format = (SkMask::Format)MASK_FORMAT_UNKNOWN;
     }
 
     glyph->fMaskFormat = format;
-    fProxy->getMetrics(glyph);
-
-    if (SkMask::kARGB32_Format == format) {
-        SkVector advance;
-        fMatrix.mapXY(SkFixedToScalar(glyph->fAdvanceX),
-                      SkFixedToScalar(glyph->fAdvanceY), &advance);
-        glyph->fAdvanceX = SkScalarToFixed(advance.fX);
-        glyph->fAdvanceY = SkScalarToFixed(advance.fY);
-
-        SkPath path;
-        fProxy->getPath(*glyph, &path);
-        path.transform(fMatrix);
-
-        SkRect storage;
-        const SkPaint& paint = fFace->paint();
-        const SkRect& newBounds = paint.doComputeFastBounds(path.getBounds(),
-                                                            &storage,
-                                                            SkPaint::kFill_Style);
-        SkIRect ibounds;
-        newBounds.roundOut(&ibounds);
-        glyph->fLeft = ibounds.fLeft;
-        glyph->fTop = ibounds.fTop;
-        glyph->fWidth = ibounds.width();
-        glyph->fHeight = ibounds.height();
-    } else {
-        SkPath      devPath, fillPath;
-        SkMatrix    fillToDevMatrix;
-
-        this->internalGetPath(*glyph, &fillPath, &devPath, &fillToDevMatrix);
-
-        // just use devPath
-        const SkIRect ir = devPath.getBounds().roundOut();
-
-        if (ir.isEmpty() || !ir.is16Bit()) {
-            glyph->fLeft    = 0;
-            glyph->fTop     = 0;
-            glyph->fWidth   = 0;
-            glyph->fHeight  = 0;
-            return;
-        }
-        glyph->fLeft    = ir.fLeft;
-        glyph->fTop     = ir.fTop;
-        glyph->fWidth   = SkToU16(ir.width());
-        glyph->fHeight  = SkToU16(ir.height());
-
-        if (glyph->fWidth > 0) {
-            switch (glyph->fMaskFormat) {
-            case SkMask::kLCD16_Format:
-                glyph->fWidth += 2;
-                glyph->fLeft -= 1;
-                break;
-            default:
-                break;
-            }
-        }
-    }
 }
 
 void SkRandomScalerContext::generateImage(const SkGlyph& glyph) {
     SkMask::Format format = (SkMask::Format)glyph.fMaskFormat;
-    switch (glyph.getGlyphID() % 3) {
+    switch (glyph.getGlyphID() % 6) {
         case 0:
+        case 1:
+        case 2:
+            break;
+        case 3:
             format = SkMask::kLCD16_Format;
             break;
-        case 1:
+        case 4:
             format = SkMask::kA8_Format;
             break;
-        case 2:
+        case 5:
             format = SkMask::kARGB32_Format;
             break;
     }
     const_cast<SkGlyph&>(glyph).fMaskFormat = format;
 
     // if the format is ARGB, we just draw the glyph from path ourselves.  Otherwise, we force
     // our proxy context to generate the image from paths.
-    if (!fFakeIt) {
-        if (SkMask::kARGB32_Format == glyph.fMaskFormat) {
-            SkPath path;
-            fProxy->getPath(glyph, &path);
+    if (SkMask::kARGB32_Format == glyph.fMaskFormat) {
+        SkPath path;
+        fProxy->getPath(glyph, &path);
 
-            SkBitmap bm;
-            bm.installPixels(SkImageInfo::MakeN32Premul(glyph.fWidth, glyph.fHeight),
-                             glyph.fImage, glyph.rowBytes());
-            bm.eraseColor(0);
+        SkBitmap bm;
+        bm.installPixels(SkImageInfo::MakeN32Premul(glyph.fWidth, glyph.fHeight),
+                         glyph.fImage, glyph.rowBytes());
+        bm.eraseColor(0);
 
-            SkCanvas canvas(bm);
-            canvas.translate(-SkIntToScalar(glyph.fLeft),
-                             -SkIntToScalar(glyph.fTop));
-            canvas.concat(fMatrix);
-            canvas.drawPath(path, fFace->paint());
-        } else {
-            fProxy->forceGenerateImageFromPath();
-            fProxy->getImage(glyph);
-            fProxy->forceOffGenerateImageFromPath();
-        }
+        SkCanvas canvas(bm);
+        canvas.translate(-SkIntToScalar(glyph.fLeft),
+                         -SkIntToScalar(glyph.fTop));
+        canvas.concat(fMatrix);
+        canvas.drawPath(path, fFace->paint());
     } else {
-        sk_bzero(glyph.fImage, glyph.computeImageSize());
+        this->forceGenerateImageFromPath();
+        fProxy->getImage(glyph);
+        this->forceOffGenerateImageFromPath();
     }
 }
 
 void SkRandomScalerContext::generatePath(const SkGlyph& glyph, SkPath* path) {
     fProxy->getPath(glyph, path);
     path->transform(fMatrix);
 }
 
 void SkRandomScalerContext::generateFontMetrics(SkPaint::FontMetrics* metrics) {
     fProxy->getFontMetrics(metrics);
     if (metrics) {
         SkScalar scale = fMatrix.getScaleY();
         metrics->fTop = SkScalarMul(metrics->fTop, scale);
         metrics->fAscent = SkScalarMul(metrics->fAscent, scale);
         metrics->fDescent = SkScalarMul(metrics->fDescent, scale);
         metrics->fBottom = SkScalarMul(metrics->fBottom, scale);
         metrics->fLeading = SkScalarMul(metrics->fLeading, scale);
         metrics->fAvgCharWidth = SkScalarMul(metrics->fAvgCharWidth, scale);
         metrics->fXMin = SkScalarMul(metrics->fXMin, scale);
         metrics->fXMax = SkScalarMul(metrics->fXMax, scale);
         metrics->fXHeight = SkScalarMul(metrics->fXHeight, scale);
     }
 }
 
 ///////////////////////////////////////////////////////////////////////////////
 
 #include "SkTypefaceCache.h"
 
-SkRandomTypeface::SkRandomTypeface(SkTypeface* proxy, const SkPaint& paint, bool fakeIt)
+SkRandomTypeface::SkRandomTypeface(SkTypeface* proxy, const SkPaint& paint)
     : SkTypeface(proxy->fontStyle(), SkTypefaceCache::NewFontID(), false)
     , fProxy(SkRef(proxy))
-    , fPaint(paint)
-    , fFakeIt(fakeIt) {}
+    , fPaint(paint) {}
 
 SkRandomTypeface::~SkRandomTypeface() {
     fProxy->unref();
 }
 
 SkScalerContext* SkRandomTypeface::onCreateScalerContext(
                                             const SkDescriptor* desc) const {
-    return SkNEW_ARGS(SkRandomScalerContext, (const_cast<SkRandomTypeface*>(this), desc, fFakeIt));
+    return SkNEW_ARGS(SkRandomScalerContext, (const_cast<SkRandomTypeface*>(this), desc));
 }
 
 void SkRandomTypeface::onFilterRec(SkScalerContextRec* rec) const {
     fProxy->filterRec(rec);
     rec->setHinting(SkPaint::kNo_Hinting);
     rec->fMaskFormat = SkMask::kARGB32_Format;
 }
 
 SkAdvancedTypefaceMetrics* SkRandomTypeface::onGetAdvancedTypefaceMetrics(
                                 PerGlyphInfo info,
@@ skipping 34 matching lines @@
 
 int SkRandomTypeface::onGetTableTags(SkFontTableTag tags[]) const {
     return fProxy->getTableTags(tags);
 }
 
 size_t SkRandomTypeface::onGetTableData(SkFontTableTag tag, size_t offset,
                                     size_t length, void* data) const {
     return fProxy->getTableData(tag, offset, length, data);
 }