Update path to Android font config file

Update path to Android font config file

In version LMP, Android is expected to change the path to their font
configuration file (and condense several files into one), as well as
changing the format (qv http://crrev.com/446473003/). This patch tries
the new path before falling back to the old path.

The new fonts are incompatible with assumptions made by the old
SkFontConfigInterfaceAndroid, so this patch MUST NOT land until
https://crrev.com/462073002/ or the equivalent has turned on the
new SkFontManager.

BUG=chromium:400801
R=djsollen@google.com,bungeman@google.com

Committed: https://skia.googlesource.com/skia/+/94fa4b99e2a53e997a90c7808cc5263f1bf40c9f

Committed: https://skia.googlesource.com/skia/+/7a4747f4f62e4896d8f8469e1939b8191fff8d4e

[tomhudson, 2014-08-08 19:35:26]

Dag-nabbit, this picked up another CL on the wrong branch. Only intended change
is the one to SkFontConfigParser_android.

Will clean up Monday.

[tomhudson, 2014-08-11 20:59:33]

Confirmed with Raph that he really did mean to put the XML in /system/etc, not
/system/fonts. Does this logic make sense?

[tomhudson, 2014-08-11 21:45:13]

While testing with SK_FONTHOST_DOES_NOT_USE_FONTMGR removed, unit tests pointed
out I'd left SkAutoAsciiToLC out of the parser; added that to this patch.

[bungeman-skia, 2014-08-12 17:54:18]

lgtm

[tomhudson, 2014-08-12 17:58:06]

The CQ bit was checked by tomhudson@google.com

[commit-bot: I haz the power, 2014-08-12 17:58:17]

CQ is trying da patch. Follow status at
 https://skia-tree-status.appspot.com/cq/tomhudson@chromium.org/458543002/80001

[commit-bot: I haz the power, 2014-08-12 18:05:40]

Change committed as 94fa4b99e2a53e997a90c7808cc5263f1bf40c9f

[djsollen, 2014-08-13 13:31:53]

late comments.

https://codereview.chromium.org/458543002/diff/80001/src/ports/SkFontConfigPa...
File src/ports/SkFontConfigParser_android.cpp (right):

https://codereview.chromium.org/458543002/diff/80001/src/ports/SkFontConfigPa...
src/ports/SkFontConfigParser_android.cpp:22: // /system/fonts/fonts.xml. Instead
of trying to detect which version
too late, but update this in a future CL to reflect the correct location.

https://codereview.chromium.org/458543002/diff/80001/src/ports/SkFontConfigPa...
src/ports/SkFontConfigParser_android.cpp:467: if (0 == fontFamilies.count()) {
seems potentially wrong to trigger on that in case the caller doesn't provide an
empty fontFamlies array

[tomhudson, 2014-08-13 17:55:13]

Derek - thanks for the additional review; patch 6 should have addressed your two
comments.

[djsollen, 2014-08-13 17:56:47]

PS6 lgtm

[tomhudson, 2014-08-13 17:57:56]

The CQ bit was checked by tomhudson@google.com

[commit-bot: I haz the power, 2014-08-13 17:58:26]

CQ is trying da patch. Follow status at
 https://skia-tree-status.appspot.com/cq/tomhudson@chromium.org/458543002/100001

[commit-bot: I haz the power, 2014-08-13 18:06:48]

Committed patchset #6 (100001) as 7a4747f4f62e4896d8f8469e1939b8191fff8d4e