From 9583ab9d7aca8590879b70c0d1738437ea163e00 Mon Sep 17 00:00:00 2001 From: reed Date: Wed, 9 Sep 2015 13:25:24 -0700 Subject: don't peek w/o first acquiring the lock BUG=skia: Review URL: https://codereview.chromium.org/1324503005 --- src/lazy/SkDiscardableMemoryPool.cpp | 7 ------- 1 file changed, 7 deletions(-) (limited to 'src/lazy') diff --git a/src/lazy/SkDiscardableMemoryPool.cpp b/src/lazy/SkDiscardableMemoryPool.cpp index 78c48f50c1..1f3bcf93e9 100644 --- a/src/lazy/SkDiscardableMemoryPool.cpp +++ b/src/lazy/SkDiscardableMemoryPool.cpp @@ -202,13 +202,6 @@ void DiscardableMemoryPool::free(PoolDiscardableMemory* dm) { bool DiscardableMemoryPool::lock(PoolDiscardableMemory* dm) { SkASSERT(dm != nullptr); - if (nullptr == dm->fPointer) { - #if SK_LAZY_CACHE_STATS - SkAutoMutexAcquire autoMutexAcquire(fMutex); - ++fCacheMisses; - #endif // SK_LAZY_CACHE_STATS - return false; - } SkAutoMutexAcquire autoMutexAcquire(fMutex); if (nullptr == dm->fPointer) { // May have been purged while waiting for lock. -- cgit v1.2.3