commit d90024d4982937c249bbbff8f02f46bfce71bb97
author Herb Derby <herb@google.com> Fri Nov 20 10:21:32 2020 -0500
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Sat Nov 21 02:24:07 2020 +0000
tree 7344308a0e3e90bef1ae3da6924ad610bdfa35cd
parent a03f8bf837e8d8b654f6a64fce55b0d3f0e10185

split subrun API into Blob and Op interfaces

Instead of using sub classing to generate the atlas API, give
the atlas portion its own interface. Then the atlas subrun
variants can subclass both interfaces.

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

src/gpu/ops/GrAtlasTextOp.cpp

diff --git a/src/gpu/ops/GrAtlasTextOp.cpp b/src/gpu/ops/GrAtlasTextOp.cpp
index 116826a..61508e4 100644
--- a/src/gpu/ops/GrAtlasTextOp.cpp
+++ b/src/gpu/ops/GrAtlasTextOp.cpp
@@ -471,7 +471,8 @@
         return nullptr;
     }
 
-    GrAtlasSubRun* subRun = static_cast<GrAtlasSubRun*>(blob->subRunList().head());
+    GrAtlasSubRun* subRun = blob->subRunList().head()->testingOnly_atlasSubRun();
+    SkASSERT(subRun);
     GrOp::Owner op;
     std::tie(std::ignore, op) = subRun->makeAtlasTextOp(nullptr, mtxProvider, glyphRunList, rtc);
     return op;