[Webkit-unassigned] [Bug 106842] adding support for DiscardablePixelRef for caching lazily decoded images
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Wed Jan 23 19:14:03 PST 2013
https://bugs.webkit.org/show_bug.cgi?id=106842
--- Comment #42 from Min Qin <qinmin at chromium.org> 2013-01-23 19:15:56 PST ---
(From update of attachment 184272)
View in context: https://bugs.webkit.org/attachment.cgi?id=184272&action=review
>> Source/WebCore/platform/graphics/chromium/DiscardablePixelRef.cpp:55
>> + delete pixelRef;
>
> instead of manually calling delete, can you put pixelRef in an OwnPtr<> and .leakPtr() it on line 49?
Done.
>> Source/WebCore/platform/graphics/chromium/DiscardablePixelRef.cpp:89
>> + m_discardable->lock();
>
> don't you need to check the return value of lock() to see if the memory's still there before calling ->data() ?
Since we have a DCHECK instead of returning a NULL on ->data() call after locking failure, we have to check this return value now. Code changed
--
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