Re: [webkit-dev] Color profiles in expected.png files

2011-09-12 Thread Peter Kasting
On Fri, Sep 9, 2011 at 7:21 PM, Simon Fraser simon.fra...@apple.com wrote:

 I think we should be consistent about color profiles, and in a way that
 doesn't break pixel tests. Does anyone want to vote one way or the other?


I suspect no profiles is the only way that will work across both ports
that handle embedded color profiles and ports that do not.  (AFAIK some
Chromium ports currently do not.)

PK
___
webkit-dev mailing list
webkit-dev@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev


Re: [webkit-dev] Color profiles in expected.png files

2011-09-10 Thread Dirk Schulze

The color profile switches automatically for me on running pixel tests with 
Lion or SL. How is it even possible that we got images with different color 
profiles for mac results?

Dirk

Simon Fraser:
 It turns out that some of the layout test expected.png files have color 
 profiles (Generic RGB), and some do not 
 https://bugs.webkit.org/show_bug.cgi?id=67880.
 
 On my Lion machine, at least, this affects the appearance of the standard 
 green color, and causes pixel tests to fail.
 
 It appears that these files have come from various sources, including Tony 
 Chang's embedding of checksums in image files, and new expected results 
 created on Lion.
 
 I think we should be consistent about color profiles, and in a way that 
 doesn't break pixel tests. Does anyone want to vote one way or the other?
 
 Once we decide, we should also have some kind of protection against adding 
 new pixel results which don't follow convention. Maybe webkit-patch can be 
 taught about this.
 
 Simon
 
 ___
 webkit-dev mailing list
 webkit-dev@lists.webkit.org
 http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev

___
webkit-dev mailing list
webkit-dev@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev


[webkit-dev] Color profiles in expected.png files

2011-09-09 Thread Simon Fraser
It turns out that some of the layout test expected.png files have color 
profiles (Generic RGB), and some do not 
https://bugs.webkit.org/show_bug.cgi?id=67880.

On my Lion machine, at least, this affects the appearance of the standard green 
color, and causes pixel tests to fail.

It appears that these files have come from various sources, including Tony 
Chang's embedding of checksums in image files, and new expected results created 
on Lion.

I think we should be consistent about color profiles, and in a way that doesn't 
break pixel tests. Does anyone want to vote one way or the other?

Once we decide, we should also have some kind of protection against adding new 
pixel results which don't follow convention. Maybe webkit-patch can be taught 
about this.

Simon

___
webkit-dev mailing list
webkit-dev@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev