Don't read unecessary font files on Android.

Don't read unecessary font files on Android.
    
With a new style fonts.xml (version >= 21) fallback fonts are specified
as font families without a name. The current code reads these, but then
also reads in the old fallback and vendor xml files. With this change,
when a new style xml file is encoutered, no further files will be read.
This both lowers memory use and speeds font lookup. Locally, this
change reduces the number of font families loaded on my 'L' device from
148 to 80. All of the families removed are duplicates.

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

[bungeman-skia, 2015-01-30 21:56:37]

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

[bungeman-skia, 2015-01-30 21:56:38]

Also updates the dumping code so I could make sure nothing changed there.

[bungeman-skia, 2015-02-02 16:32:04]

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

[bungeman-skia, 2015-02-02 16:32:04]

Now that the other changes have landed, this one can.

[bungeman-skia, 2015-02-02 16:42:47]

The CQ bit was checked by bungeman@google.com

[commit-bot: I haz the power, 2015-02-02 16:43:11]

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

[commit-bot: I haz the power, 2015-02-02 16:43:12]

Note for Reviewers:
The CQ is waiting for an approval.
If you believe that the CL is not ready yet, or if you would like to L-G-T-M
with comments then please uncheck the CQ checkbox.

Waiting for LGTM from valid reviewer(s) till 2015-02-02 22:43 UTC

[mtklein, 2015-02-02 16:55:49]

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

https://codereview.chromium.org/888923003/diff/40001/src/ports/SkFontConfigPa...
src/ports/SkFontConfigParser_android.cpp:427: static int
appendSystemFontFamilies(SkTDArray<FontFamily*> &fontFamilies) {
Let's update to static_function_name_style and T& foo while we're here?

[bungeman-skia, 2015-02-02 18:36:37]

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

https://codereview.chromium.org/888923003/diff/40001/src/ports/SkFontConfigPa...
src/ports/SkFontConfigParser_android.cpp:427: static int
appendSystemFontFamilies(SkTDArray<FontFamily*> &fontFamilies) {
On 2015/02/02 16:55:49, mtklein wrote:
> Let's update to static_function_name_style and T& foo while we're here?

Done.

[bungeman-skia, 2015-02-02 18:40:18]

The CQ bit was checked by bungeman@google.com

[commit-bot: I haz the power, 2015-02-02 18:40:52]

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

[commit-bot: I haz the power, 2015-02-02 18:40:53]

Note for Reviewers:
The CQ is waiting for an approval.
If you believe that the CL is not ready yet, or if you would like to L-G-T-M
with comments then please uncheck the CQ checkbox.

Waiting for LGTM from valid reviewer(s) till 2015-02-03 00:40 UTC

[mtklein, 2015-02-02 18:45:58]

lgtm

[commit-bot: I haz the power, 2015-02-02 19:19:48]

Committed patchset #5 (id:80001) as
https://skia.googlesource.com/skia/+/efbad37180c6b6f90d4b7e96a234e8065d2ec395