Remove getGlyphIDAdvance() from SkStrike's API

Change-Id: Id9648dfc44ec7bc7f37a281236ff5fc756f6e646
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/215610
Commit-Queue: Herb Derby <herb@google.com>
Reviewed-by: Mike Klein <mtklein@google.com>

Author: herbderby

src/core/SkStrike.h

@@ -45,11 +45,6 @@ class SkStrike final : public SkStrikeInterface {
     /**  Return a glyph that has no information if it is not already filled out. */
     SkGlyph* getRawGlyphByID(SkPackedGlyphID);
 
-    /** Returns a glyph with valid fAdvance and fDevKern fields. The remaining fields may be
-        valid, but that is not guaranteed. If you require those, call getGlyphIDMetrics instead.
-    */
-    const SkGlyph& getGlyphIDAdvance(SkGlyphID);
-
     /** Returns a glyph with all fields valid except fImage and fPath, which may be null. If they
         are null, call findImage or findPath for those. If they are not null, then they are valid.
 
@@ -187,6 +182,11 @@ class SkStrike final : public SkStrikeInterface {
         kHashMask  = kHashCount - 1
     };
 
+    /** Returns a glyph with valid fAdvance field. The remaining fields may be
+        valid, but that is not guaranteed. If you require those, call getGlyphIDMetrics instead.
+    */
+    const SkGlyph& getGlyphIDAdvance(SkGlyphID);
+
     // Return the SkGlyph* associated with MakeID. The id parameter is the
     // combined glyph/x/y id generated by MakeID. If it is just a glyph id
     // then x and y are assumed to be zero. Limit the amount of work using type.