commit d13487e71ba1e0c0a6d312c1c42f6b660464bc0b
author Ben Wagner <bungeman@google.com> Wed Jun 03 23:29:43 2020 -0400
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Thu Jun 04 16:45:09 2020 +0000
tree 27409ed85fdd5ed443f6a4db592591f54ddd5100
parent 06f3d088e1211d384c55443a9f05f25477005366

Regularize SkFontMgr for CoreText build.

Move SkFontHost_mac.cpp into an 'optional' like the other fontmgr build
rules. This allows building with other fontmgrs on Mac and makes the lib
dependencies explicit. In the future this helps with splitting the out
the default factory.

Change-Id: Iecef9e428acb69f89b54afa00b4e779f5858c61a
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/294076
Reviewed-by: Mike Klein <mtklein@google.com>
Commit-Queue: Ben Wagner <bungeman@google.com>

samplecode/SampleTextBox.cpp

diff --git a/samplecode/SampleTextBox.cpp b/samplecode/SampleTextBox.cpp
index c13b981..72bc272 100644
--- a/samplecode/SampleTextBox.cpp
+++ b/samplecode/SampleTextBox.cpp
@@ -114,7 +114,7 @@
 };
 
 DEF_SAMPLE( return new TextBoxView([](){ return SkShaper::Make(); }, "default"); );
-#ifdef SK_BUILD_FOR_MAC
+#ifdef SK_SHAPER_CORETEXT_AVAILABLE
 DEF_SAMPLE( return new TextBoxView(SkShaper::MakeCoreText, "coretext"); );
 #endif
 
@@ -176,7 +176,9 @@
         for (SkScalar size = 30; size <= 30; size += 10) {
             this->drawTest(canvas, text, size, SkShaper::Make());
             canvas->translate(0, size + 5);
+            #ifdef SK_SHAPER_CORETEXT_AVAILABLE
             this->drawTest(canvas, text, size, SkShaper::MakeCoreText());
+            #endif
             canvas->translate(0, size*2);
         }
     }