commit 70cd6d14465cb89a9ebb22e2286a1ab0fce9b582
author Herb Derby <herb@google.com> Wed Feb 24 09:14:36 2021 -0500
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Wed Feb 24 15:51:54 2021 +0000
tree f4ed2533999de4a9e87a64a1c487a8cfc941e219
parent e40e69f3a45b9c9db634f0efabcfbe35eaea3446

make producing GrTextBlob sub runs thread safe

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

src/gpu/GrSurfaceDrawContext.cpp

diff --git a/src/gpu/GrSurfaceDrawContext.cpp b/src/gpu/GrSurfaceDrawContext.cpp
index dff2642..3803946 100644
--- a/src/gpu/GrSurfaceDrawContext.cpp
+++ b/src/gpu/GrSurfaceDrawContext.cpp
@@ -429,18 +429,15 @@
             textBlobCache->add(glyphRunList, blob);
         }
 
-        // TODO(herb): redo processGlyphRunList to handle shifted draw matrix.
         bool supportsSDFT = fContext->priv().caps()->shaderCaps()->supportsDistanceFieldText();
-        for (auto& glyphRun : glyphRunList) {
-            fGlyphPainter.processGlyphRun(glyphRun,
-                                          viewMatrix.localToDevice(),
-                                          drawOrigin,
-                                          drawPaint,
-                                          fSurfaceProps,
-                                          supportsSDFT,
-                                          options,
-                                          blob.get());
-        }
+        blob->makeSubRuns(&fGlyphPainter,
+                          glyphRunList,
+                          viewMatrix.localToDevice(),
+                          drawOrigin,
+                          drawPaint,
+                          fSurfaceProps,
+                          supportsSDFT,
+                          options);
     }
 
     for (const GrSubRun& subRun : blob->subRunList()) {