Bug 66231 - record90-paint-alpha-clip*.xfail.png are not needed
Summary: record90-paint-alpha-clip*.xfail.png are not needed
Status: RESOLVED INVALID
Alias: None
Product: cairo
Classification: Unclassified
Component: general (show other bugs)
Version: 1.12.14
Hardware: Other All
: medium normal
Assignee: Chris Wilson
QA Contact: cairo-bugs mailing list
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-06-26 23:53 UTC by Bryce Harrington
Modified: 2013-06-27 16:31 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments
0001-test-Drop-redundant-record90-paint-alpha-clip-xfail-.patch (2.71 KB, patch)
2013-06-26 23:59 UTC, Bryce Harrington
Details | Splinter Review

Description Bryce Harrington 2013-06-26 23:53:08 UTC
The xfail files record90-paint-alpha-clip-mask.xfail.png and
record90-paint-alpha-clip.xfail.png override reference images
record90-paint-alpha-clip-mask.*.ref.png and
record90-paint-alpha-clip.*.ref.png, respectively.  Yet the differences
in these files is quite minor, even if we turn the threshold down to
just a pixel:


record90-paint-alpha-clip.base.argb32.ref.png
record90-paint-alpha-clip.xfail.png
PASS: Images are binary identical

record90-paint-alpha-clip.base.rgb24.ref.png
record90-paint-alpha-clip.xfail.png
PASS: Images are binary identical

record90-paint-alpha-clip.ref.png record90-paint-alpha-clip.xfail.png
PASS: Images are perceptually indistinguishable

record90-paint-alpha-clip-mask.base.argb32.ref.png
record90-paint-alpha-clip-mask.xfail.png
FAIL: Images are visibly different
4 pixels are different

record90-paint-alpha-clip-mask.base.rgb24.ref.png
record90-paint-alpha-clip-mask.xfail.png
FAIL: Images are visibly different
4 pixels are different

record90-paint-alpha-clip-mask.ref.png
record90-paint-alpha-clip-mask.xfail.png
PASS: Images are perceptually indistinguishable
Comment 1 Bryce Harrington 2013-06-26 23:59:31 UTC
Created attachment 81516 [details] [review]
0001-test-Drop-redundant-record90-paint-alpha-clip-xfail-.patch

The two xfail.png's are unnecessary and can be safely dropped.  Dropping one of them results in a handful of test cases PASSing, that had been XFAIL previously.  The other was already PASSing, but only because the diff was within the threshold; with the XFAIL gone the test now gets an exact match.
Comment 2 Chris Wilson 2013-06-27 07:28:17 UTC
You've underestimated the severity of any discrepancy in the record tests. (And I fudged the base ref pngs.)
Comment 3 Uli Schlachter 2013-06-27 16:28:55 UTC
(In reply to comment #2)
> You've underestimated the severity of any discrepancy in the record tests.
> (And I fudged the base ref pngs.)

So does this mean that e.g. record90-paint-alpha-clip.base.argb32.ref.png is wrong and the identical file record90-paint-alpha-clip.xfail.png just works around this wrongness? Because in this cases there is no discrepancy at all (according to Bryce).
Comment 4 Chris Wilson 2013-06-27 16:31:41 UTC
Yes, copying the base output as its ref image in this case was my mistake.


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.