diff options
| author |  commit-bot@chromium.org <commit-bot@chromium.org@2bbb7eff-a529-9590-31e7-b0007b416f81> | 2013-10-24 17:44:27 +0000 |
|---|---|---|
| committer | commit-bot@chromium.org <commit-bot@chromium.org@2bbb7eff-a529-9590-31e7-b0007b416f81> | 2013-10-24 17:44:27 +0000 |
| commit | 50a3043194cf278a74ff51c33c6cdb52cbe1f8f9 (patch) | |
| tree | 7749fd49ae609c1e3f0f9d770510b667477eae94 /src/gpu/GrResourceCache.h | |
| parent | 81c67001f2486be220462a29538606112825b817 (diff) | |
We want to give SkPixelRef a way to signal over to GrResourceCache that it's become pointless to keep around textures based on that SkPixelRef when its pixels change, so that it can be a good citizen and free those textures.
This adds an invalidation listener mechanism to SkPixelRef to let it send this message while still staying ignorant of who's listening.
These messages are tricky to deliver. The SkPixelRefs they originates from and the GrResourceCaches they ultimately end up at may be on different threads; neither class is threadsafe; their object lifetimes are totally independent; it's a many-senders-to-many-receivers relation; and neither codebase should really know about the other.
So I've added a per-message-type global message bus to broadcast messages to threadsafe inboxes. Anyone can post() a message, which will show up in all the inboxes of that type, read whenever the inbox's owner calls poll(). The implementation is _dumb_; it can be improved in several dimensions (inbox size limits, lock-free message delivery) if we find the need.
I took some care to make sure not to send the invalidation message for any SkPixelRef that's sharing a generation ID with another SkPixelRef.
BUG=
R=bsalomon@google.com, scroggo@google.com, reed@google.com
Author: mtklein@google.com
Review URL: https://codereview.chromium.org/26734003
git-svn-id: http://skia.googlecode.com/svn/trunk@11949 2bbb7eff-a529-9590-31e7-b0007b416f81
Diffstat (limited to 'src/gpu/GrResourceCache.h')
| -rw-r--r-- | src/gpu/GrResourceCache.h | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/src/gpu/GrResourceCache.h b/src/gpu/GrResourceCache.h index a9adf91b6f..38378ac771 100644 --- a/src/gpu/GrResourceCache.h +++ b/src/gpu/GrResourceCache.h @@ -15,6 +15,7 @@ #include "GrTypes.h" #include "GrTHashTable.h" #include "GrBinHashKey.h" +#include "SkMessageBus.h" #include "SkTInternalLList.h" class GrResource; @@ -141,6 +142,11 @@ private: Key fKey; }; +// The cache listens for these messages to purge junk resources proactively. +struct GrResourceInvalidatedMessage { + GrResourceKey key; +}; + /////////////////////////////////////////////////////////////////////////////// class GrResourceEntry { @@ -395,6 +401,10 @@ private: void internalPurge(int extraCount, size_t extraBytes); + // Listen for messages that a resource has been invalidated and purge cached junk proactively. + SkMessageBus<GrResourceInvalidatedMessage>::Inbox fInvalidationInbox; + void purgeInvalidated(); + #ifdef SK_DEBUG static size_t countBytes(const SkTInternalLList<GrResourceEntry>& list); #endif |