[Webkit-unassigned] [Bug 106842] adding support for DiscardablePixelRef for caching lazily decoded images

[bugzilla-daemon at webkit.org]

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





--- Comment #63 from Hin-Chung Lam <hclam at google.com>  2013-01-25 15:46:20 PST ---
(From update of attachment 184817)
View in context: https://bugs.webkit.org/attachment.cgi?id=184817&action=review

> Source/WebCore/platform/graphics/chromium/ImageDecodingStore.cpp:245
> +            if (!cacheEntry->useCount() && (!cacheEntry->isDiscardable() || !m_cacheLimitInBytes))

I don't understand the condition for !m_cacheLimitInBytes. When will it be zero? It seems to be not necessary.

> Source/WebCore/platform/graphics/chromium/ImageDecodingStore.cpp:251
> +        removeFromCacheListInternal(heapCacheEntriesToDelete);

I think this line is wrong.

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