Fix refcounting bug for Type1 font usage with more than 255 characters.

src/pdf/SkPDFFont.cpp

Index: src/pdf/SkPDFFont.cpp
diff --git a/src/pdf/SkPDFFont.cpp b/src/pdf/SkPDFFont.cpp
index b824506b189715b5ab0f04255716597c95482ae0..f4a543985de9aa7c3f6402b3283a4daf82a57bd3 100644
--- a/src/pdf/SkPDFFont.cpp
+++ b/src/pdf/SkPDFFont.cpp
@@ -878,6 +878,7 @@ SkPDFFont::SkPDFFont(SkAdvancedTypefaceMetrics* info, SkTypeface* typeface,
           fFontInfo(info),

[bungeman-skia, 2013/12/05 18:36:20]

fFontInfo(SkSafeRef(info)),
fDescriptor(SkSafeRef(relatedFontDescriptor))

here makes this a bit shorter and makes it clear in one place that these two are
ref owning.

[vandebo (ex-Chrome), 2013/12/05 19:34:56]

Done.  Indeed, I didn't realize that SkSafeRef returned the obj.

           fDescriptor(relatedFontDescriptor) {
     SkSafeRef(info);
+    SkSafeRef(relatedFontDescriptor);
     if (info == NULL) {
         fFontType = SkAdvancedTypefaceMetrics::kNotEmbeddable_Font;
     } else if (info->fMultiMaster) {