remove firstSubRun

Since exposing subRunList, there is no reason to have the additional API
call for getting the head of the list.

Change-Id: Ib4bf8b7b89484154ad8d576bb5397810628f1a11
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/302258
Commit-Queue: Herb Derby <herb@google.com>
Reviewed-by: Robert Phillips <robertphillips@google.com>
Auto-Submit: Herb Derby <herb@google.com>
This commit is contained in:
Herb Derby 2020-07-13 10:22:32 -04:00 committed by Skia Commit-Bot
parent 2c180304dc
commit 342273cbaa
3 changed files with 2 additions and 5 deletions

View File

@ -501,13 +501,13 @@ std::unique_ptr<GrDrawOp> GrAtlasTextOp::CreateOpTestingOnly(GrRenderTargetConte
glyphRunList, drawMatrix, rtc->surfaceProps(),
contextPriv.caps()->shaderCaps()->supportsDistanceFieldText(),
SDFOptions, blob.get());
if (!blob->firstSubRun()) {
if (!blob->subRunList().head()) {
return nullptr;
}
std::unique_ptr<GrDrawOp> op;
std::tie(std::ignore, op) =
blob->firstSubRun()->makeAtlasTextOp(nullptr, mtxProvider, glyphRunList, rtc);
blob->subRunList().head()->makeAtlasTextOp(nullptr, mtxProvider, glyphRunList, rtc);
return op;
}

View File

@ -896,4 +896,3 @@ void GrTextBlob::processSourceMasks(const SkZip<SkGlyphVariant, SkPoint>& drawab
this->addMultiMaskFormat(SubRun::MakeTransformedMask, drawables, strikeSpec);
}
auto GrTextBlob::firstSubRun() const -> SubRun* { return fSubRunList.head(); }

View File

@ -125,8 +125,6 @@ public:
const SkZip<SkGlyphVariant, SkPoint>& drawables,
const SkStrikeSpec& strikeSpec);
SubRun* firstSubRun() const;
const SkTInternalLList<SubRun>& subRunList() const { return fSubRunList; }
private: