Speed up GrResourceCache add and lookup by using TDynamicHash

include/gpu/GrResource.h

Index: include/gpu/GrResource.h
diff --git a/include/gpu/GrResource.h b/include/gpu/GrResource.h
index 93dec58d80632a2bea38303c3a32be994e73f237..4d9951655ec09d902c0808372123670f34a2c572 100644
--- a/include/gpu/GrResource.h
+++ b/include/gpu/GrResource.h
@@ -84,6 +84,8 @@ public:
 
     void setNeedsDeferredUnref() { fFlags |= kDeferredUnref_FlagBit; }
 
+    SK_DECLARE_NAMED_INTERNAL_LLIST_INTERFACE(GrResource, CacheLRU);
+    SK_DECLARE_NAMED_INTERNAL_LLIST_INTERFACE(GrResource, CacheEntryResources);
 protected:
     /**
      * isWrapped indicates we have wrapped a client-created backend resource in a GrResource. If it
@@ -111,6 +113,9 @@ private:
 
     // We're in an internal doubly linked list
     SK_DECLARE_INTERNAL_LLIST_INTERFACE(GrResource);

[mtklein, 2013/12/03 19:02:02]

It seems weird to have one default and two named lists.  Can we just used named
ones for each?  Please update the comments to remind us how each of the three
lists matters.

Isn't the default list used as LRU?  I'm confused that this new list is named
CacheLRU.

+    SK_DECLARE_NAMED_INTERNAL_LLIST_INTERFACE_DATA(GrResource, CacheLRU);
+    SK_DECLARE_NAMED_INTERNAL_LLIST_INTERFACE_DATA(GrResource, CacheEntryResources);
+
 
     GrGpu*              fGpu;               // not reffed. The GrGpu can be deleted while there
                                             // are still live GrResources. It will call