DM FontMgr updates

  - return nullptr for the various makeFromFoo() that we can't support,
    and tweak a few unit tests to bail out early when they do

  - create FontStyleSet and SkTypefaces once

  - abort early from FontHostStream if we can't openStream()

  - implement SkTestTypeface::onCreateFamilyNameIterator()
    with SkOTUtils::LocalizedStrings_SingleName() so FontNames passes

  - pin out-of-range glyph IDs to zero in SkTestTypeface

Change-Id: Iac53265e331fc1c5c507513af3ab299063e6610a
Reviewed-on: https://skia-review.googlesource.com/69501
Reviewed-by: Ben Wagner <bungeman@google.com>
Commit-Queue: Mike Klein <mtklein@chromium.org>

1 files changed, 9 insertions, 1 deletions

diff --git a/tests/TypefaceTest.cpp b/tests/TypefaceTest.cpp
index 6175d36157..c8e96c5a7c 100644
--- a/tests/TypefaceTest.cpp
+++ b/tests/TypefaceTest.cpp
@@ -50,7 +50,10 @@ static void TypefaceStyle_test(skiatest::Reporter* reporter,
     os2Table->usWidthClass.value = static_cast<WidthType>(SkEndian_SwapBE16(width));
 
     sk_sp<SkTypeface> newTypeface(SkTypeface::MakeFromStream(new SkMemoryStream(sk_ref_sp(data))));
-    SkASSERT_RELEASE(newTypeface);
+    if (!newTypeface) {
+        // Not all SkFontMgr can MakeFromStream().
+        return;
+    }
 
     SkFontStyle newStyle = newTypeface->fontStyle();
 
@@ -111,6 +114,11 @@ DEF_TEST(TypefaceAxes, reporter) {
     // TODO: if axes are set and the back-end doesn't support them, should we create the typeface?
     sk_sp<SkTypeface> typeface = fm->makeFromStream(std::move(distortable), params);
 
+    if (!typeface) {
+        // Not all SkFontMgr can makeFromStream().
+        return;
+    }
+
     int count = typeface->getVariationDesignPosition(nullptr, 0);
     if (count == -1) {
         return;