commit 249b40e9de049464b1d78b0df88bf291fdb4f293
author Herbert Derby <herb@google.com> Tue Feb 12 10:36:35 2019 -0500
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Tue Feb 12 16:38:30 2019 +0000
tree eec00681173a03807d5881960d52c8e45bae7e2a
parent 0b6ae6386d34efa7554d1b515c016f1c66e09a30

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>

src/core/SkStrike.cpp

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);