Move SkTypeface to sk_sp.

gm/colortype.cpp

 /*
  * Copyright 2013 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #include "gm.h"
 #include "SkCanvas.h"
 #include "SkGradientShader.h"
 #include "../src/fonts/SkGScalerContext.h"
 
 class ColorTypeGM : public skiagm::GM {
 public:
-    ColorTypeGM()
-        : fColorType(nullptr) {
-    }
-
-    virtual ~ColorTypeGM() {
-        SkSafeUnref(fColorType);
-    }
+    ColorTypeGM() {}
 
 protected:
     void onOnceBeforeDraw() override {
         const SkColor colors[] = {
             SK_ColorRED, SK_ColorGREEN, SK_ColorBLUE,
             SK_ColorMAGENTA, SK_ColorCYAN, SK_ColorYELLOW
         };
         SkMatrix local;
         local.setRotate(180);
         SkPaint paint;
         paint.setAntiAlias(true);
         paint.setShader(SkGradientShader::MakeSweep(0, 0, colors, nullptr, SK_ARRAY_COUNT(colors),
                                                     0, &local));
 
-        SkTypeface* orig = sk_tool_utils::create_portable_typeface("serif", SkTypeface::kBold);
+        auto orig = sk_tool_utils::create_portable_typeface("serif", SkTypeface::kBold);

[f(malita), 2016/05/02 13:46:35]

Nit: Chrome disallows auto for smart ptrs, and I think we're trying to stick to
the same rule in Skia these days.

[bungeman-skia, 2016/05/02 20:24:54]

These were all in the CL I started this with... will remove since I don't like
them either. Done.

[reed1, 2016/05/03 13:45:00]

I didn't know that. I'd like to be caught up in that discussion.

         if (nullptr == orig) {
-            orig = SkTypeface::RefDefault();
+            orig = SkTypeface::MakeDefault();
         }
-        fColorType = new SkGTypeface(orig, paint);
-        orig->unref();
+        fColorType = sk_make_sp<SkGTypeface>(std::move(orig), paint);
     }
 
     SkString onShortName() override {
         return SkString("colortype");
     }
 
     SkISize onISize() override {
         return SkISize::Make(640, 480);
     }
 
     void onDraw(SkCanvas* canvas) override {
         SkPaint paint;
         paint.setAntiAlias(true);
         paint.setTypeface(fColorType);
 
         for (SkScalar size = 10; size <= 100; size += 10) {
             paint.setTextSize(size);
             canvas->translate(0, paint.getFontMetrics(nullptr));
             canvas->drawText("Hamburgefons", 12, 10, 10, paint);
         }
     }
 
 private:
-    SkTypeface* fColorType;
+    sk_sp<SkTypeface> fColorType;
 
     typedef skiagm::GM INHERITED;
 };
 
 DEF_GM(return new ColorTypeGM;)