Use SkTArray instead of SkSTArray in movable.

Use SkTArray instead of SkSTArray in movable.

SkTArray is trivially movable, but SkSTArray is not, since it may
contain an internal pointer. FontFileInfo is supposed to be
trivially movable but currently isn't due to containing an SkSTArray.
Removing this particular optimization should not affect performance
much as this type is only used for parsing and does not allocate any
memory unless a variation axis is actually specified.

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

[bungeman-skia, 2015-05-27 14:43:27]

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

[bungeman-skia, 2015-05-27 14:43:28]

https://codereview.chromium.org/1155383002/diff/1/src/ports/SkFontConfigParse...
File src/ports/SkFontConfigParser_android.h (right):

https://codereview.chromium.org/1155383002/diff/1/src/ports/SkFontConfigParse...
src/ports/SkFontConfigParser_android.h:79: SkTArray<Axis, true> fAxes;
Note comment above that FontFileInfo must remain trivially movable.

[bungeman-skia, 2015-05-27 14:44:42]

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

[commit-bot: I haz the power, 2015-05-27 14:44:52]

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

[commit-bot: I haz the power, 2015-05-27 14:50:23]

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

[commit-bot: I haz the power, 2015-05-27 14:50:23]

Dry run: This issue passed the CQ dry run.

[reed1, 2015-05-27 20:08:48]

lgtm

[bungeman-skia, 2015-05-27 20:09:05]

The CQ bit was checked by bungeman@google.com

[commit-bot: I haz the power, 2015-05-27 20:09:38]

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

[commit-bot: I haz the power, 2015-05-27 20:10:06]

Committed patchset #1 (id:1) as
https://skia.googlesource.com/skia/+/a9f1db7902227143f97e4e29c1eefcd690f6aced