[Webkit-unassigned] [Bug 93467] Change ImageSource to be asynchronous.

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Aug 15 19:31:53 PDT 2012


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





--- Comment #24 from Huang Dongsung <luxtella at company100.net>  2012-08-15 19:32:25 PST ---
(In reply to comment #21)
> Please limit the scope of this change to just refactoring ImageSource to be asynchronous, i.e. limit the scope to just ImageSource::requestFrameAtIndex(), ImageSourceObserver, SynchronousImageSource and switch usage of ImageSource to SynchronousImageSource.
> 
> We should do the refactoring of ImageSource, ImageDecoder::decodeAtIndex in a separate patch and discussed in a different bug. Having it included in this change is confusing for other reviewers. I think your second patch with a few nits would be good for review.

Thank you for your review. I agree with you. I'll file another bug about ImageDecoder.

-- 
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