Trim whitespace from parsed filename in Android v21.
authorbungeman <bungeman@google.com>
Fri, 8 May 2015 15:31:54 +0000 (08:31 -0700)
committerCommit bot <commit-bot@chromium.org>
Fri, 8 May 2015 15:31:54 +0000 (08:31 -0700)
commitc0727d117e67844f0c8794cc7eaa49a96a015347
tree6c3aeacdf0700ca9b08f33bfff4f6f42e02e368b
parent72c7a60ecfb9519580373b4641be25af97d081e8
Trim whitespace from parsed filename in Android v21.

The entire text content of the 'font' element is currently used as the
file name. This wasn't an issue in earlier versions, as the 'file'
element was dedicated to only containing the file name. The new 'font'
element also contains a number of attributes and potentially other tags.
This means that a 'font' element can become quite long, making it
desireable to be able to split it across multiple lines.

However, splitting the 'font' element across multiple lines is currently
difficult and awkward as any whitespace outside of tags will be
considered part of the file name. This change means that any leading or
trailing whitespace will not be considered part of the file name.

This only applies to v21 and later files, so while this restricts font
file names from beginning and ending with whitespace, it is unlikely to
break any users in practice. It is probably also undesireable to have
font files with names that begin or end with whitespace in any event.

Review URL: https://codereview.chromium.org/1125413003
resources/android_fonts/v22/fonts.xml
src/ports/SkFontConfigParser_android.cpp