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

[bugzilla-daemon at webkit.org]

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





--- Comment #69 from Huang Dongsung <luxtella at company100.net>  2012-08-02 20:24:20 PST ---
(In reply to comment #67)
> I think just unplugging image decoding from currentFrameHasAlpha() can get us going with with everything except multiframe ICO and GIF already. The benefit of supporting ICO and GIF seems small given the complexity to do it correctly.

I agreed on you, so parallel image decoders don't support ICO and GIF.

> 
> That's why I back off from removing decoding from frameIsCompleteAtIndex() and frameDurationAtIndex().

I think we can make it clear when to start image decoding.
After we discussed, I think there are only two methods which should trigger decoding: BitmapImage::frameAtIndex and BitmapImage::startAnimation, not frameIsCompleteAtIndex() and frameDurationAtIndex().

If we don't make frameIsCompleteAtIndex() not trigger image decoding, I'm afraid that other developers will be easy to make a mistake to use frameIsCompleteAtIndex() and it can cause to stop parallel image decoding.

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