Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(141)

Unified Diff: src/gpu/GrTextStrike_impl.h

Issue 390103002: Replace use of GrTHashTable in GrFontCache with SkTDynamicHash. (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Remove deleteAll(); use iterator instead. Created 6 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « src/gpu/GrTextStrike.cpp ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/gpu/GrTextStrike_impl.h
diff --git a/src/gpu/GrTextStrike_impl.h b/src/gpu/GrTextStrike_impl.h
index 1dd623d24cddf32634fd04177432559e1901b4df..db1bf27cef0bca30a62c065c28b5fe79bdf7f912 100644
--- a/src/gpu/GrTextStrike_impl.h
+++ b/src/gpu/GrTextStrike_impl.h
@@ -11,25 +11,6 @@
#ifndef GrTextStrike_impl_DEFINED
#define GrTextStrike_impl_DEFINED
-class GrFontCache::Key {
-public:
- explicit Key(const GrFontDescKey* fontScalarKey) {
- fFontScalerKey = fontScalarKey;
- }
-
- intptr_t getHash() const { return fFontScalerKey->getHash(); }
-
- static bool LessThan(const GrTextStrike& strike, const Key& key) {
- return *strike.getFontScalerKey() < *key.fFontScalerKey;
- }
- static bool Equals(const GrTextStrike& strike, const Key& key) {
- return *strike.getFontScalerKey() == *key.fFontScalerKey;
- }
-
-private:
- const GrFontDescKey* fFontScalerKey;
-};
-
void GrFontCache::detachStrikeFromList(GrTextStrike* strike) {
if (strike->fPrev) {
SkASSERT(fHead != strike);
@@ -51,10 +32,9 @@ void GrFontCache::detachStrikeFromList(GrTextStrike* strike) {
GrTextStrike* GrFontCache::getStrike(GrFontScaler* scaler, bool useDistanceField) {
this->validate();
- const Key key(scaler->getKey());
- GrTextStrike* strike = fCache.find(key);
+ GrTextStrike* strike = fCache.find(*(scaler->getKey()));
if (NULL == strike) {
- strike = this->generateStrike(scaler, key);
+ strike = this->generateStrike(scaler);
} else if (strike->fPrev) {
// Need to put the strike at the head of its dllist, since that is how
// we age the strikes for purging (we purge from the back of the list)
@@ -72,27 +52,6 @@ GrTextStrike* GrFontCache::getStrike(GrFontScaler* scaler, bool useDistanceField
///////////////////////////////////////////////////////////////////////////////
-/**
- * This Key just wraps a glyphID, and matches the protocol need for
- * GrTHashTable
- */
-class GrTextStrike::Key {
-public:
- Key(GrGlyph::PackedID id) : fPackedID(id) {}
-
- uint32_t getHash() const { return fPackedID; }
-
- static bool LessThan(const GrGlyph& glyph, const Key& key) {
- return glyph.fPackedID < key.fPackedID;
- }
- static bool Equals(const GrGlyph& glyph, const Key& key) {
- return glyph.fPackedID == key.fPackedID;
- }
-
-private:
- GrGlyph::PackedID fPackedID;
-};
-
GrGlyph* GrTextStrike::getGlyph(GrGlyph::PackedID packed,
GrFontScaler* scaler) {
GrGlyph* glyph = fCache.find(packed);
« no previous file with comments | « src/gpu/GrTextStrike.cpp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698