Allow Android font manager on *nix.

Allow Android font manager on *nix.

There is nothing left in the SkFontMgr_android.cpp which is specific
to Android, so allow it to be built and used on *nix platforms. This
allows for easier development and testing.

The only reason to limit to *nix platforms are the dependencies on
Expat and FreeType. This should be buildable and runnable on other
platforms when these dependencies are also available.

Committed: https://skia.googlesource.com/skia/+/786ece17c3fac1aae670a6206ad4d11471a2488e

[bungeman-skia, 2015-07-13 20:45:44]

The CQ bit was checked by bungeman@google.com to run a CQ dry run

[commit-bot: I haz the power, 2015-07-13 20:46:04]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1228833003/1

[commit-bot: I haz the power, 2015-07-13 20:53:28]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-07-13 20:53:29]

Dry run: This issue passed the CQ dry run.

[bungeman-skia, 2015-07-13 20:56:26]

bungeman@google.com changed reviewers:
+ djsollen@google.com

[bungeman-skia, 2015-07-13 20:56:26]

This is a step toward building and testing all the FreeType based font managers
on all platforms.

[djsollen, 2015-07-15 13:48:48]

https://codereview.chromium.org/1228833003/diff/1/gyp/ports.gyp
File gyp/ports.gyp (right):

https://codereview.chromium.org/1228833003/diff/1/gyp/ports.gyp#newcode71
gyp/ports.gyp:71: 
remove newline

https://codereview.chromium.org/1228833003/diff/1/gyp/ports.gyp#newcode133
gyp/ports.gyp:133: 'sources/': [['include',
'../src/ports/SkFontMgr_android_factory.cpp']],
I've never seen this syntax before.  What does it do special that you don't get
by just having 'sources': [ '...' ]?

[bungeman-skia, 2015-07-15 17:08:48]

https://codereview.chromium.org/1228833003/diff/1/gyp/ports.gyp
File gyp/ports.gyp (right):

https://codereview.chromium.org/1228833003/diff/1/gyp/ports.gyp#newcode71
gyp/ports.gyp:71: 
On 2015/07/15 13:48:48, djsollen wrote:
> remove newline

Done.

https://codereview.chromium.org/1228833003/diff/1/gyp/ports.gyp#newcode133
gyp/ports.gyp:133: 'sources/': [['include',
'../src/ports/SkFontMgr_android_factory.cpp']],
On 2015/07/15 13:48:48, djsollen wrote:
> I've never seen this syntax before.  What does it do special that you don't
get
> by just having 'sources': [ '...' ]?

These '/' suffix on the key means inclusion/exclusion using regex. Since you
must have only one '_factory' (or you'll get linker errors) the most obvious way
was to list them, exclude them all above (see line 74) and then re-include
exactly one of them based on conditions. Once excluded the only way to bring
them back is to later include them, so that's what happening here. This CL isn't
really introducing this, it's just moving the existing ones around (see 220 in
the original for where this came from).

Also, as soething of a side note, gyp internally provides sources_excluded to
generators, so that IDEs can still show excluded sources. If a source is
actually removed, I don't think it will show up that way.

[djsollen, 2015-07-15 17:23:32]

lgtm

[bungeman-skia, 2015-07-15 17:44:10]

The CQ bit was checked by bungeman@google.com

[commit-bot: I haz the power, 2015-07-15 17:44:26]

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

[commit-bot: I haz the power, 2015-07-15 17:58:50]

Committed patchset #2 (id:20001) as
https://skia.googlesource.com/skia/+/786ece17c3fac1aae670a6206ad4d11471a2488e