|Summary:||WebPImageDecoder: Increase image/webp decoding performance 10-20%|
|Product:||WebKit||Reporter:||noel gordon <noel.gordon>|
|Component:||New Bugs||Assignee:||noel gordon <noel.gordon>|
|Severity:||Normal||CC:||noel.gordon, pascal.massimino, webkit.review.bot|
|Version:||528+ (Nightly build)|
|Bug Depends on:|
Description noel gordon 2011-12-11 20:29:08 PST
Data copies from RGB m_rgbBuffer to the pixel backing store, via buffer.setRGBA(), are costly.
Comment 2 WebKit Review Bot 2011-12-12 00:10:52 PST
Comment on attachment 118725 [details] Patch Clearing flags on attachment: 118725 Committed r102561: <http://trac.webkit.org/changeset/102561>
Comment 3 WebKit Review Bot 2011-12-12 00:10:56 PST
All reviewed patches have been landed. Closing bug.
Comment 4 Pascal Massimino 2011-12-12 03:18:53 PST
lgtm. Thanks for doing this! This means: every incoming features (alpha, etc.) will have to be thought-out mainly from the incremental perspective.
Comment 5 noel gordon 2011-12-13 22:15:19 PST
s/mainly/eventually/, if alpha images did not initially support progressive decodes, I'd use "allDataReceived" in the code. Of course early in decoding setup, where grab the image size say, is where I'd want to find out if the image has alpha.