Closed Bug 453547 Opened 16 years ago Closed 16 years ago

reftest reference pngs for ACID2 test and APNG have embedded color profiles

Categories

(Core :: Graphics: Color Management, defect)

Product:
Core
Component:
Graphics: Color Management
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED

People

(Reporter: bholley, Assigned: bholley)

References

Details

Attachments

(1 file, 1 obsolete file)

updated fix to handle apng issue also
(deleted), patch
bholley
: review+
Details | Diff | Splinter Review
For some reason our reference PNG has an embedded profile called "Color LCD". This seems like an accident, and it breaks the reftest when color management is turned on. I'll put up a patch with the profile stripped off in a few hours.
Attached patch fix (obsolete) (deleted) — — Splinter Review
patch to strip the profile. Reftest passes with CM on and off now.
Attachment #336749 - Flags: review?(vladimir)
Summary: reftest reference png for ACID2 test has an embedded color profile → reftest reference pngs for ACID2 test and APNG have embedded color profiles
Problem also found to exist for APNG test. Adding a fix for that. Vlad told me just to r it myself.
Attachment #336749 - Attachment is obsolete: true
Attachment #336754 - Flags: review?(bholley)
Attachment #336749 - Flags: review?(vladimir)
Attachment #336754 - Flags: review?(bholley) → review+
pushed in f3216177949d.
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Product: Core → Core Graveyard
Component: GFX → GFX: Color Management
Product: Core Graveyard → Core
QA Contact: general → color-management
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: