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

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

Committed: http://code.google.com/p/skia/source/detail?r=12520

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

https://codereview.chromium.org/103423003/diff/1/src/pdf/SkPDFFont.cpp
File src/pdf/SkPDFFont.cpp (right):

https://codereview.chromium.org/103423003/diff/1/src/pdf/SkPDFFont.cpp#newcod...
src/pdf/SkPDFFont.cpp:878: fFontInfo(info),
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]

https://codereview.chromium.org/103423003/diff/1/src/pdf/SkPDFFont.cpp
File src/pdf/SkPDFFont.cpp (right):

https://codereview.chromium.org/103423003/diff/1/src/pdf/SkPDFFont.cpp#newcod...
src/pdf/SkPDFFont.cpp:878: fFontInfo(info),
On 2013/12/05 18:36:20, bungeman1 wrote:
> 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.

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

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

lgtm

[commit-bot: I haz the power, 2013-12-05 20:53:00]

CQ is trying da patch. Follow status at
https://skia-tree-status.appspot.com/cq/vandebo@chromium.org/103423003/20001

[commit-bot: I haz the power, 2013-12-05 21:14:04]

Change committed as 12520