Use a consistent type for arrays of SpvId.

Previously, we had different sections of code that would use
stack<SpvId>, vector<SpvId> or SkTArray<SpvId> to represent an array
of IDs. This is both hard to read and wasteful of code size. We now
use SkTArray throughout.

stack<SpvId> is a particularly significant offender as this actually
uses deque<SpvId> internally; deques are much, much less efficient
than vectors and take a surprising amount of size on disk. Performance
is not a major issue here, but we're happy to win back codesize, which
claims a ~2K improvement: http://screen/6Vy5kRKcLb7zM5D

Change-Id: I8e46ba10298a6b9753463c3df989533e82bcf2af
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/573964
Commit-Queue: John Stiles <johnstiles@google.com>
Auto-Submit: John Stiles <johnstiles@google.com>
Reviewed-by: Herb Derby <herb@google.com>
2 files changed