diff options
| author |  bungeman <bungeman@google.com> | 2016-04-29 15:05:02 -0700 |
|---|---|---|
| committer |  Commit bot <commit-bot@chromium.org> | 2016-04-29 15:05:02 -0700 |
| commit | 572f879b6163e1a4fb790a7f63401a6cf0ade1d2 (patch) | |
| tree | 90ba38ddc87c7a119506546e6b6054ad3d3d04fd /src/ports/SkFontHost_FreeType_common.h | |
| parent | 4d9ff62fa21326003cf5394cd1d16f4b4633102a (diff) | |
Fix race in SkTypeface_FreeType::onCountGlyphs.
Found by TSAN. This makes the operation slightly slower, but it is
not on any user's hot path.
BUG=skia:5238
Review-Url: https://codereview.chromium.org/1940613002
Diffstat (limited to 'src/ports/SkFontHost_FreeType_common.h')
| -rw-r--r-- | src/ports/SkFontHost_FreeType_common.h | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/src/ports/SkFontHost_FreeType_common.h b/src/ports/SkFontHost_FreeType_common.h index 8c13a80aa5..6d050cdd99 100644 --- a/src/ports/SkFontHost_FreeType_common.h +++ b/src/ports/SkFontHost_FreeType_common.h @@ -73,7 +73,6 @@ public: protected: SkTypeface_FreeType(const SkFontStyle& style, SkFontID uniqueID, bool isFixedPitch) : INHERITED(style, uniqueID, isFixedPitch) - , fGlyphCount(-1) {} virtual SkScalerContext* onCreateScalerContext(const SkScalerContextEffects&, @@ -95,8 +94,6 @@ protected: size_t length, void* data) const override; private: - mutable int fGlyphCount; - typedef SkTypeface INHERITED; }; |