refactor emboldenGlyph

bench/FontScalerBench.cpp

 /*

[bungeman-skia, 2013/12/16 16:33:36]

I'm unsure of what the changes to this file are for. Nowhere is
paint.setFakeBoldText(true) called, and this is a bench (not a GM) so changes to
the output canvas are not tested.

[zheng.xu, 2013/12/17 10:02:51]

There are GM cases covers different font styles.
I will revert the changes on this file, since it tries to cover the code path
that won't be used later.

  * Copyright 2011 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #include "SkBenchmark.h"
 #include "SkCanvas.h"
 #include "SkGraphics.h"
 #include "SkPaint.h"
 #include "SkRandom.h"
 #include "SkString.h"
+#include "SkTypeface.h"
 
 extern bool gSkSuppressFontCachePurgeSpew;
 
+static const struct {
+    const char*       fStyleString;
+    SkTypeface::Style fStyle;
+} gStyleNames [] = {
+    { "normal", SkTypeface::kNormal },
+    { "bold", SkTypeface::kBold },
+    { "italic", SkTypeface::kItalic },
+    { "bolditalic", SkTypeface::kBoldItalic },
+};
+
 class FontScalerBench : public SkBenchmark {
-    SkString fName;
-    SkString fText;
-    bool     fDoLCD;
+    SkString          fName;
+    SkString          fText;
+    SkTypeface::Style fStyle;
+    bool              fDoLCD;
 public:
-    FontScalerBench(bool doLCD)  {
-        fName.printf("fontscaler_%s", doLCD ? "lcd" : "aa");
+    FontScalerBench(SkTypeface::Style style, bool doLCD)  {
+        const char* styleString;
+
+        styleString = gStyleNames[0].fStyleString;
+        fStyle = gStyleNames[0].fStyle;
+
+        for (unsigned int i = 0;
+            i < sizeof(gStyleNames) / sizeof(gStyleNames[0]); i++) {
+            if (gStyleNames[i].fStyle == style) {
+                styleString = gStyleNames[i].fStyleString;
+                fStyle = style;
+                break;
+            }
+        }
+
+        fName.printf("fontscaler_%s_%s", styleString, doLCD ? "lcd" : "aa");
         fText.set("abcdefghijklmnopqrstuvwxyz01234567890");
+
         fDoLCD = doLCD;
     }
 
 protected:
     virtual const char* onGetName() { return fName.c_str(); }
     virtual void onDraw(const int loops, SkCanvas* canvas) {
         SkPaint paint;
         this->setupPaint(&paint);
+
+        SkTypeface* face = SkTypeface::CreateFromName(NULL, fStyle);

[bungeman-skia, 2013/12/16 16:33:36]

SkAutoTUnref<SkTypeface> face(SkTypeface::CreateFromName(NULL, fStyle));

and remove the SkSafeUnref below.

[zheng.xu, 2013/12/17 10:02:51]

Thanks for the hint. This is quite convenient.

+        paint.setTypeface(face);
         paint.setLCDRenderText(fDoLCD);
 
         bool prev = gSkSuppressFontCachePurgeSpew;
         gSkSuppressFontCachePurgeSpew = true;
 
         for (int i = 0; i < loops; i++) {
             // this is critical - we want to time the creation process, so we
             // explicitly flush our cache before each run
             SkGraphics::PurgeFontCache();
 
-            for (int ps = 9; ps <= 24; ps += 2) {
+            for (int y = 20, ps = 9; ps <= 24; y += ps, ps += 2) {
                 paint.setTextSize(SkIntToScalar(ps));
                 canvas->drawText(fText.c_str(), fText.size(),
-                        0, SkIntToScalar(20), paint);
+                        0, SkIntToScalar(y), paint);
             }
         }
 
         gSkSuppressFontCachePurgeSpew = prev;
+
+        SkSafeUnref(face);
     }
 private:
     typedef SkBenchmark INHERITED;
 };
 
 ///////////////////////////////////////////////////////////////////////////////
 
-DEF_BENCH( return SkNEW_ARGS(FontScalerBench, (false)); )
-DEF_BENCH( return SkNEW_ARGS(FontScalerBench, (true)); )
+DEF_BENCH( return SkNEW_ARGS(FontScalerBench, (SkTypeface::kNormal, false)); )
+DEF_BENCH( return SkNEW_ARGS(FontScalerBench, (SkTypeface::kBold, false)); )
+DEF_BENCH( return SkNEW_ARGS(FontScalerBench, (SkTypeface::kItalic, false)); )
+DEF_BENCH( return SkNEW_ARGS(FontScalerBench, (SkTypeface::kBoldItalic, false)); )
+DEF_BENCH( return SkNEW_ARGS(FontScalerBench, (SkTypeface::kNormal, true)); )
+DEF_BENCH( return SkNEW_ARGS(FontScalerBench, (SkTypeface::kBold, true)); )
+DEF_BENCH( return SkNEW_ARGS(FontScalerBench, (SkTypeface::kItalic, true)); )
+DEF_BENCH( return SkNEW_ARGS(FontScalerBench, (SkTypeface::kBoldItalic, true)); )