User-Agent: Opera/9.63 (X11; Linux i686; U; en) Presto/2.1.1 Build Identifier: When CMS is enabled in tagged-only mode, the JPEG decoder does not use the direct YCC->ARGB path even for images where it could be used (ie. images without an ICC profile). Reproducible: Always
Created attachment 373630 [details] [diff] [review] Fix for the issue Average execution time for my JPEG loading benchmark case goes down from 414.67sec to 399.10sec on a 1GHz SnapDragon (ARMv7) with this patch.
Comment on attachment 373630 [details] [diff] [review] Fix for the issue Nice catch. Do you need someone to check this in for you?
Attachment #373630 - Flags: review+
(In reply to comment #2) > Do you need someone to check this in for you? Yes please.
Marking as checkin-needed.
And removing checkin-needed; I'll just check this in myself when my commit access request goes through. Do we need to get another review of this, since the r+ is a year old? JOEDREW, what do you think?
If the patch still applies and/or only has some fuzz, check it in!
Status: NEW → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.