Move SkTypeface to sk_sp.

src/ports/SkFontMgr_android.cpp

Index: src/ports/SkFontMgr_android.cpp
diff --git a/src/ports/SkFontMgr_android.cpp b/src/ports/SkFontMgr_android.cpp
index 4aea047cdeee106cbffc401d4217fb638d8d0ad4..3e6e0da18ae3913febe5f6819f30e9ab940ee779 100644
--- a/src/ports/SkFontMgr_android.cpp
+++ b/src/ports/SkFontMgr_android.cpp
@@ -371,7 +371,7 @@ protected:
     {
         for (int i = 0; i < fallbackNameToFamilyMap.count(); ++i) {
             SkFontStyleSet_Android* family = fallbackNameToFamilyMap[i].styleSet;
-            SkAutoTUnref<SkTypeface_AndroidSystem> face(family->matchStyle(style));
+            sk_sp<SkTypeface_AndroidSystem> face(family->matchStyle(style));
 
             if (!langTag.isEmpty() && !face->fLang.getTag().startsWith(langTag.c_str())) {
                 continue;
@@ -387,9 +387,6 @@ protected:
 
             uint16_t glyphID;
             paint.textToGlyphs(&character, sizeof(character), &glyphID);
-            if (glyphID != 0) {

[bungeman-skia, 2016/05/12 16:47:41]

How did this sneak in here? Without this test this method is just 'return
nullptr;'!

-                return face.release();
-            }
         }
         return nullptr;
     }