Return POD from generateMetrics() rather than mutate SkGlyph

This is a step towards making scalercontext subclasses *not* friends.
By returning a 'public' struct, we can hide write-access to fields
on SkGlyph. Other CLs will follow to complete this transitions
(e.g. generateImage will need to be tweaked).

Bug: skia:14629
Change-Id: I4bb65e1da7e4926a8b084e67fd478bbd9cfc21f3
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/727156
Auto-Submit: Mike Reed <mike@reedtribe.org>
Commit-Queue: Ben Wagner <bungeman@google.com>
Reviewed-by: Ben Wagner <bungeman@google.com>
17 files changed