commit c8e31ea399eab60bcc09e78f183c312aef709a42
author Herb Derby <herb@google.com> Mon Mar 08 12:42:17 2021 -0500
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Mon Mar 08 18:50:59 2021 +0000
tree 2319aebfb0753941deac060c233b971b1799e668
parent a085cd82636a3afe04a084a1bbe04650a61482de

GrTextBlob: remove unused hash function; add key() api

Change-Id: I20e4f2168713cbd54c0428c60e38a736cf735df3
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/381476
Reviewed-by: Brian Osman <brianosman@google.com>
Commit-Queue: Herb Derby <herb@google.com>

src/gpu/text/GrTextBlobCache.cpp

diff --git a/src/gpu/text/GrTextBlobCache.cpp b/src/gpu/text/GrTextBlobCache.cpp
index 3fbe5c5..d11a7f3 100644
--- a/src/gpu/text/GrTextBlobCache.cpp
+++ b/src/gpu/text/GrTextBlobCache.cpp
@@ -50,7 +50,7 @@
 }
 
 void GrTextBlobCache::internalRemove(GrTextBlob* blob) {
-    auto  id      = GrTextBlob::GetKey(*blob).fUniqueID;
+    auto  id      = blob->key().fUniqueID;
     auto* idEntry = fBlobIDCache.find(id);
 
     if (idEntry != nullptr) {
@@ -140,13 +140,13 @@
 }
 
 sk_sp<GrTextBlob> GrTextBlobCache::internalAdd(sk_sp<GrTextBlob> blob) {
-    auto  id      = GrTextBlob::GetKey(*blob).fUniqueID;
+    auto  id      = blob->key().fUniqueID;
     auto* idEntry = fBlobIDCache.find(id);
     if (!idEntry) {
         idEntry = fBlobIDCache.set(id, BlobIDCacheEntry(id));
     }
 
-    if (sk_sp<GrTextBlob> alreadyIn = idEntry->find(GrTextBlob::GetKey(*blob)); alreadyIn) {
+    if (sk_sp<GrTextBlob> alreadyIn = idEntry->find(blob->key()); alreadyIn) {
         blob = std::move(alreadyIn);
     } else {
         fBlobList.addToHead(blob.get());
@@ -168,17 +168,17 @@
 
 void GrTextBlobCache::BlobIDCacheEntry::addBlob(sk_sp<GrTextBlob> blob) {
     SkASSERT(blob);
-    SkASSERT(GrTextBlob::GetKey(*blob).fUniqueID == fID);
-    SkASSERT(!this->find(GrTextBlob::GetKey(*blob)));
+    SkASSERT(blob->key().fUniqueID == fID);
+    SkASSERT(!this->find(blob->key()));
 
     fBlobs.emplace_back(std::move(blob));
 }
 
 void GrTextBlobCache::BlobIDCacheEntry::removeBlob(GrTextBlob* blob) {
     SkASSERT(blob);
-    SkASSERT(GrTextBlob::GetKey(*blob).fUniqueID == fID);
+    SkASSERT(blob->key().fUniqueID == fID);
 
-    auto index = this->findBlobIndex(GrTextBlob::GetKey(*blob));
+    auto index = this->findBlobIndex(blob->key());
     SkASSERT(index >= 0);
 
     fBlobs.removeShuffle(index);
@@ -191,7 +191,7 @@
 
 int GrTextBlobCache::BlobIDCacheEntry::findBlobIndex(const GrTextBlob::Key& key) const {
     for (int i = 0; i < fBlobs.count(); ++i) {
-        if (GrTextBlob::GetKey(*fBlobs[i]) == key) {
+        if (fBlobs[i]->key() == key) {
             return i;
         }
     }