commit d0fa3ee55c4af1bc3b533cfca4fda53ab8c1c753
author Herb Derby <herb@google.com> Tue Jun 23 12:23:51 2020 -0400
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Thu Jun 25 15:32:21 2020 +0000
tree e43ac7251f0af181780520bfa57dce85687be79e
parent e88320baec76ff472d32ccd4dfcccb0a1d10eef7

add mutex to GrTextBlobCache

This should be the bulk of the work. The next part is getting it to live
in a common place, and wiring up the callbacks properly.

The callback points are fPurgeMore and fMessageBusID.

Change-Id: Ibabe285ace8a0b6531572b755a6c4f7bd41b1f6c
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/298400
Commit-Queue: Herb Derby <herb@google.com>
Reviewed-by: Robert Phillips <robertphillips@google.com>

src/gpu/text/GrTextBlobCache.cpp

diff --git a/src/gpu/text/GrTextBlobCache.cpp b/src/gpu/text/GrTextBlobCache.cpp
index f981b59..8a329c4 100644
--- a/src/gpu/text/GrTextBlobCache.cpp
+++ b/src/gpu/text/GrTextBlobCache.cpp
@@ -27,17 +27,20 @@
                                 const SkMatrix& viewMatrix) {
     sk_sp<GrTextBlob> cacheBlob(GrTextBlob::Make(glyphRunList, viewMatrix));
     cacheBlob->setupKey(key, blurRec, glyphRunList.paint());
+    SkAutoMutexExclusive lock{fMutex};
     this->internalAdd(cacheBlob);
     glyphRunList.temporaryShuntBlobNotifyAddedToCache(fMessageBusID);
     return cacheBlob;
 }
 
 sk_sp<GrTextBlob> GrTextBlobCache::find(const GrTextBlob::Key& key) const {
+    SkAutoMutexExclusive lock{fMutex};
     const auto* idEntry = fBlobIDCache.find(key.fUniqueID);
     return idEntry ? idEntry->find(key) : nullptr;
 }
 
 void GrTextBlobCache::remove(GrTextBlob* blob) {
+    SkAutoMutexExclusive lock{fMutex};
     this->internalRemove(blob);
 }
 
@@ -55,6 +58,7 @@
 }
 
 void GrTextBlobCache::makeMRU(GrTextBlob* blob) {
+    SkAutoMutexExclusive lock{fMutex};
     if (fBlobList.head() == blob) {
         return;
     }
@@ -64,12 +68,14 @@
 }
 
 void GrTextBlobCache::freeAll() {
+    SkAutoMutexExclusive lock{fMutex};
     fBlobIDCache.reset();
     fBlobList.reset();
     fCurrentSize = 0;
 }
 
 void GrTextBlobCache::setBudget(size_t budget) {
+    SkAutoMutexExclusive lock{fMutex};
     fSizeBudget = budget;
     this->internalCheckPurge();
 }
@@ -80,6 +86,7 @@
 }
 
 void GrTextBlobCache::purgeStaleBlobs() {
+    SkAutoMutexExclusive lock{fMutex};
     this->internalPurgeStaleBlobs();
 }
 
@@ -106,6 +113,7 @@
 }
 
 size_t GrTextBlobCache::usedBytes() const {
+    SkAutoMutexExclusive lock{fMutex};
     return fCurrentSize;
 }