RESOLVED FIXED 42463
[Qt] Avoid QImage::pixel() in getImageData() 
https://bugs.webkit.org/show_bug.cgi?id=42463
Summary [Qt] Avoid QImage::pixel() in getImageData()
Andreas Kling
Reported 2010-07-16 09:36:18 PDT
From the QImage documentation: Warning: This function is expensive when used for massive pixel manipulations. We should use scanLine() (or even better in Qt 4.7, constScanLine()) instead.
Attachments
Proposed patch (1.56 KB, patch)
2010-07-16 09:38 PDT, Andreas Kling 		no flags
DetailsFormatted DiffDiff
Proposed patch v2 (1.54 KB, patch)
2010-07-16 13:01 PDT, Andreas Kling 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Andreas Kling
Comment 1 2010-07-16 09:38:02 PDT
Created attachment 61817 [details] Proposed patch
Andreas Kling
Comment 2 2010-07-16 09:38:53 PDT
This change yields a 1% speedup on http://mrdoob.com/lab/javascript/effects/plane_deformations/
Andreas Kling
Comment 3 2010-07-16 13:01:48 PDT
Created attachment 61840 [details] Proposed patch v2 Oops, don't use QRgb* to traverse the scan lines, since QRgb might be 64-bit. Use quint32* instead.
Andreas Kling
Comment 4 2010-07-19 05:45:58 PDT
Comment on attachment 61840 [details] Proposed patch v2 Clearing flags on attachment: 61840 Committed r63652: <http://trac.webkit.org/changeset/63652>
Andreas Kling
Comment 5 2010-07-19 05:46:08 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.