RESOLVED FIXED 30740
[Qt] QtWebKit should consider using QImageReader::setQuality(49) 
https://bugs.webkit.org/show_bug.cgi?id=30740
Summary [Qt] QtWebKit should consider using QImageReader::setQuality(49)
Holger Freyther
Reported 2009-10-23 23:35:07 PDT
ImageDecoderQt.cpp should use QImageReader::setQuality(49) to use JDCT_IFAST in the Qt JPEG Image Decoder. This change can give a 5% speedup on the image_cycling reduction. JDCT_IFAST is supposingly not generating as good results as the other integer based method but it is not certain how bad these are. The Palm developers have opted to use this for the WebOS.
Attachments
Add attachment proposed patch, testcase, etc.
Yael
Comment 1 2009-10-30 05:04:07 PDT
(In reply to comment #0) > ImageDecoderQt.cpp should use QImageReader::setQuality(49) to use JDCT_IFAST in > the Qt JPEG Image Decoder. This change can give a 5% speedup on the > image_cycling reduction. > > JDCT_IFAST is supposingly not generating as good results as the other integer > based method but it is not certain how bad these are. The Palm developers have > opted to use this for the WebOS. Sorry I did not get back to you in that. I was OOO until today. Will try to arrange testing during next week.
Yael
Comment 2 2009-10-30 14:22:23 PDT
We collected a few JPG images and tested on N97. The colors of the images looked better after we applied this change.
Holger Freyther
Comment 3 2009-12-14 02:51:34 PST
I landed a patch to do this in r52086. Let us see if someone is noticing the difference.
Note You need to log in before you can comment on or make changes to this bug.