Remove redundant fGlyphMap.set
allocateNewGlyph calls fGlyphMap.set. No reason to call it again in lookupPackedGlyphID.
Change-Id: I3724094de6a76a9c5e9f11a1808b14c3dd50b64e
Reviewed-on: https://skia-review.googlesource.com/c/191561
Auto-Submit: Herb Derby <herb@google.com>
Commit-Queue: Ben Wagner <bungeman@google.com>
Reviewed-by: Ben Wagner <bungeman@google.com>
diff --git a/src/core/SkStrike.cpp b/src/core/SkStrike.cpp
index 20be04b..436f734 100644
--- a/src/core/SkStrike.cpp
+++ b/src/core/SkStrike.cpp
@@ -93,7 +93,6 @@
if (nullptr == glyphPtr) {
glyphPtr = this->allocateNewGlyph(packedGlyphID, type);
- fGlyphMap.set(glyphPtr);
} else {
if (type == kFull_MetricsType && glyphPtr->isJustAdvance()) {
fScalerContext->getMetrics(glyphPtr);