[Webkit-unassigned] [Bug 82287] MemoryCache should adopt our standard RefCounted model for object lifetime
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Tue May 22 16:06:19 PDT 2012
https://bugs.webkit.org/show_bug.cgi?id=82287
Eric Seidel <eric at webkit.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #143386|review?, commit-queue? |review+, commit-queue+
Flag| |
--- Comment #15 from Eric Seidel <eric at webkit.org> 2012-05-22 16:05:16 PST ---
(From update of attachment 143386)
LGTM. Please add some explanation to the top of CachedResource.h as well as CachedResourceHandle.h to explain when one is supposed to use a CachedResourceHandle, vs. a RefPtr. You can do that in a separate patch, but I believe it's important to document this distinction in the code now that it's possible to confuse these two.
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the webkit-unassigned
mailing list