commit 3d6bf04366f6bb33b058b39066dd92334c185a66
author Herb Derby <herb@google.com> Fri Jun 12 11:09:48 2020 -0400
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Fri Jun 12 17:05:22 2020 +0000
tree a9d0e0bd1b4c46a494e7a55af844c314a8a785be
parent 2af055b14d709731340e4fa8cb80843fc8ddb531

make options robust

Options used to rely on an external call to SanitizeOptions to
ensure they were correct. Now that defaults are set directly,
make sure that the values coming in make sense.

Change-Id: If6cfc027722b6a7717a920b482ec5be8f7526367
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/296040
Reviewed-by: Robert Phillips <robertphillips@google.com>
Commit-Queue: Herb Derby <herb@google.com>

src/gpu/ops/GrAtlasTextOp.cpp

diff --git a/src/gpu/ops/GrAtlasTextOp.cpp b/src/gpu/ops/GrAtlasTextOp.cpp
index 85cc6cc..c900866 100644
--- a/src/gpu/ops/GrAtlasTextOp.cpp
+++ b/src/gpu/ops/GrAtlasTextOp.cpp
@@ -561,11 +561,7 @@
     auto glyphRunList = builder.useGlyphRunList();
 
     const GrRecordingContextPriv& contextPriv = rtc->fContext->priv();
-    GrTextContext::Options SDFOptions = {
-            contextPriv.options().fMinDistanceFieldFontSize,
-            contextPriv.options().fGlyphsAsPathsFontSize
-    };
-    GrTextContext::SanitizeOptions(&SDFOptions);
+    GrTextContext::Options SDFOptions = rtc->fContext->priv().SDFTOptions();
 
     if (glyphRunList.empty()) {
         return nullptr;