Closed Bug 489142 Opened 12 years ago Closed 11 years ago

JPEG decoder does not use fastpath when CMS mode is tagged-only

Categories

(Core :: ImageLib, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: ilpo.ruotsalainen, Assigned: ilpo.ruotsalainen)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

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
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+
Status: UNCONFIRMED → NEW
Ever confirmed: true
(In reply to comment #2)
> Do you need someone to check this in for you?

Yes please.
Marking as checkin-needed.
Assignee: nobody → ilpo.ruotsalainen
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?
Keywords: checkin-needed
If the patch still applies and/or only has some fuzz, check it in!
Attachment #373630 - Flags: approval2.0?
Attachment #373630 - Flags: approval2.0? → approval2.0+
http://hg.mozilla.org/mozilla-central/rev/3c5313a9c437
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.