Extend SkFontMgr_Custom to cover ttc, otf, pfb.

Extend SkFontMgr_Custom to cover ttc, otf, pfb.

Adds ability to test ScanFont with Type1 (pfb) fonts
and fonts with multiple faces (ttc).

Committed: https://skia.googlesource.com/skia/+/ee0c2e4fd429424beaa35f29e7f656997ba3f115

Committed: https://skia.googlesource.com/skia/+/14df8339841f246a7337a8fb0d90f1b7ee689619

[bungeman-skia, 2014-10-28 16:41:35]

bungeman@google.com changed reviewers:
+ halcanary@google.com, reed@google.com

[bungeman-skia, 2014-10-28 16:41:36]

The biggest change here is to SkFontHost_linux.cpp which halcanary is most
interested in.

The motivation for this was to allow for a simple test harness for debugging
issues with Type1 fonts, which can be difficult at times. This makes it easier
as one can gyp with '-Dno_skia_fontconfig' on Linux and then only load '.pfb'
files.

The second motivation was that after I wrote this I recognized that here and on
Android we're creating and destroying FT_Library objects like they're going out
of style.

[reed1, 2014-10-28 16:49:32]

lgtm

[hal.canary, 2014-10-28 16:50:15]

On 2014/10/28 16:41:36, bungeman1 wrote:
> The biggest change here is to SkFontHost_linux.cpp which halcanary is most
> interested in.

No bots are using this anymore, and might not until we port Skia to a new
architecture - or any other system lacking a stable freetype.

[bungeman-skia, 2014-10-28 17:30:12]

On 2014/10/28 16:50:15, Hal Canary wrote:
> On 2014/10/28 16:41:36, bungeman1 wrote:
> > The biggest change here is to SkFontHost_linux.cpp which halcanary is most
> > interested in.
> 
> No bots are using this anymore, and might not until we port Skia to a new
> architecture - or any other system lacking a stable freetype.

Thanks for pointing that out, I was looking for the bot since I know one existed
at some point. I've built this locally myself, so at least it's been tested that
way.

[bungeman-skia, 2014-10-28 17:32:27]

The CQ bit was checked by bungeman@google.com

[commit-bot: I haz the power, 2014-10-28 17:33:21]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/672723002/20001

[commit-bot: I haz the power, 2014-10-28 17:46:11]

Committed patchset #2 (id:20001) as ee0c2e4fd429424beaa35f29e7f656997ba3f115

[bungeman-skia, 2014-10-28 19:03:04]

A revert of this CL (patchset #2 id:20001) has been created in
https://codereview.chromium.org/685803003/ by bungeman@google.com.

The reason for reverting is: Reverting as this appears to be causing issues with
color fonts on Android..

[bungeman-skia, 2014-10-28 21:58:04]

The CQ bit was checked by bungeman@google.com

[commit-bot: I haz the power, 2014-10-28 21:58:16]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/672723002/40001

[commit-bot: I haz the power, 2014-10-28 22:07:28]

Committed patchset #3 (id:40001) as 14df8339841f246a7337a8fb0d90f1b7ee689619