[Webkit-unassigned] [Bug 90375] Parallel image decoders

[bugzilla-daemon at webkit.org]

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





--- Comment #64 from Hin-Chung Lam <hclam at google.com>  2012-08-02 17:02:01 PST ---
(From update of attachment 156058)
View in context: https://bugs.webkit.org/attachment.cgi?id=156058&action=review

> Source/WebCore/platform/graphics/ImageSource.cpp:191
> +    if (fileExt == "ico" || fileExt == "gif")

I looked again to see if it is possible to make BitmapImage::frameDurationAtIndex() and BitmapImage::frameIsCompleteAtIndex() not trigger image decoding. It proved to be very difficult to do so correctly. I think we're much safer to leave these two methods untouched. This is okay because they are used only with GIF animation which you excluded from parallel image decoding already.

To loosen the restriction of this line and still have the correct behavior is to check for frameCount(). If the frameCount() == 1 it's okay to be GIF and ICO.

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