Make GrFontScaler not be ref-counted

src/gpu/text/GrTextUtils.cpp

Index: src/gpu/text/GrTextUtils.cpp
diff --git a/src/gpu/text/GrTextUtils.cpp b/src/gpu/text/GrTextUtils.cpp
index c368c214e18692c633404df51c54e8d7ef1f14ba..da2b3e20dbbcde7bb309244b874e8abac1418dc7 100644
--- a/src/gpu/text/GrTextUtils.cpp
+++ b/src/gpu/text/GrTextUtils.cpp
@@ -572,7 +572,7 @@ uint32_t GrTextUtils::FilterTextFlags(const SkSurfaceProps& surfaceProps, const
 
 static void glyph_cache_aux_proc(void* data) {
     GrFontScaler* scaler = (GrFontScaler*)data;
-    SkSafeUnref(scaler);
+    delete scaler;
 }
 
 GrFontScaler* GrTextUtils::GetGrFontScaler(SkGlyphCache* cache) {

[bungeman-skia, 2016/05/17 14:14:05]

It actually seems strange that this is in GrTextUtils as well, since this is the
only valid way to call the constructor for GrFontScaler (it's a factory). It
seems like this function should be named GrFontScaler::Get(SkGlyphCache* strike)
and the GrFontScaler constructor be made private.