[Webkit-unassigned] [Bug 82287] MemoryCache should adopt our standard RefCounted model for object lifetime

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=82287





--- Comment #13 from Nate Chapin <japhet at chromium.org>  2012-05-22 15:20:47 PST ---
(In reply to comment #12)
> I see.  So basically just replacing the m_handleCount with a real m_refCount.  The question to me is what happens when things hold onto a CachedResource w/o a full-blown handle.  Will things get confused.

I think the rule for now is "If you aren't SubresourceLoader and you don't live in WebCore/lodaer/cache/, you need a CachedResourceHandle instead of a RefPtr for a class level variable".

I'm pondering how to do away with CachedResourceHandle entirely, but I think that's something to deal with after this bug.

> 
> If you could upload a patch which fixes the EWS I'm happy to take a second, longer look.

Will do.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.