diff options
author | reed <reed@google.com> | 2014-09-15 11:39:44 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2014-09-15 11:39:45 -0700 |
commit | c90e0149ec530075cae7bf51072a16628311855e (patch) | |
tree | d34427f8d88f6ca1b3563214e38eb6f568e199bc /bench/ImageCacheBench.cpp | |
parent | 327f905d2cb0d37c302d651d8f2b17ea56368467 (diff) |
Change SkResourceCache to take a Visitor inside its find().
This reverts commit 595aa05efcb504e85358b8d328ac4a9fa1c46e2e.
BUG=skia:
R=mtklein@google.com, danakj@chromium.org
Author: reed@google.com
Review URL: https://codereview.chromium.org/569353002
Diffstat (limited to 'bench/ImageCacheBench.cpp')
-rw-r--r-- | bench/ImageCacheBench.cpp | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/bench/ImageCacheBench.cpp b/bench/ImageCacheBench.cpp index ca2b9342d5..0f8fdf2708 100644 --- a/bench/ImageCacheBench.cpp +++ b/bench/ImageCacheBench.cpp @@ -27,6 +27,10 @@ struct TestRec : public SkResourceCache::Rec { virtual const Key& getKey() const SK_OVERRIDE { return fKey; } virtual size_t bytesUsed() const SK_OVERRIDE { return sizeof(fKey) + sizeof(fValue); } + + static bool Visitor(const SkResourceCache::Rec&, void*) { + return true; + } }; } @@ -41,7 +45,7 @@ public: void populateCache() { for (int i = 0; i < CACHE_COUNT; ++i) { - fCache.unlock(fCache.addAndLock(SkNEW_ARGS(TestRec, (TestKey(i), i)))); + fCache.add(SkNEW_ARGS(TestRec, (TestKey(i), i))); } } @@ -58,8 +62,8 @@ protected: TestKey key(-1); // search for a miss (-1) for (int i = 0; i < loops; ++i) { - SkDEBUGCODE(SkResourceCache::ID id =) fCache.findAndLock(key); - SkASSERT(NULL == id); + SkDEBUGCODE(bool found =) fCache.find(key, TestRec::Visitor, NULL); + SkASSERT(!found); } } |