Bug 16244 – DRT doesn't handle platform specific pixel test results correctly

• Status: RESOLVED
• Resolution: FIXED
• Component: Tools / Tests

[Nikolas Zimmermann]

As the topic says we have a bug with platform specific pixel test results
SVG wants cross-platform *-expected.txt results, but platform specific *-expected.png files - living in platform/<yourplatform>/svg/...

Currently on every pixel test run DRT recreates the png files in svg/W3C-SVG-1.1 instead of platform/..
It compares correctly though against the PNGs living in platform/.

[Nikolas Zimmermann]

Note: I was wrong, it does not compare correctly! It compares against the new generated pixel test results :(

[Nikolas Zimmermann]

Okay, I have a fix for this. It highlights that we're missing several layout test pixel results in trunk:

?      svg/carto.net/frameless-svg-parse-error-expected.checksum
?      svg/carto.net/frameless-svg-parse-error-expected.png
?      svg/css/getComputedStyle-basic-expected.checksum
?      svg/css/getComputedStyle-basic-expected.png
?      svg/css/glyph-orientation-rounding-test-expected.checksum
?      svg/css/glyph-orientation-rounding-test-expected.png
?      svg/custom/SVGException-expected.checksum
?      svg/custom/SVGException-expected.png
?      svg/custom/embedded-svg-disallowed-in-dashboard-expected.checksum
?      svg/custom/embedded-svg-disallowed-in-dashboard-expected.png
?      svg/custom/getBBox-js-expected.checksum
?      svg/custom/getBBox-js-expected.png
?      svg/custom/image-with-prefix-in-webarchive-expected.checksum
?      svg/custom/image-with-prefix-in-webarchive-expected.png
?      svg/custom/manually-parsed-embedded-svg-disallowed-in-dashboard-expected.checksum
?      svg/custom/manually-parsed-embedded-svg-disallowed-in-dashboard-expected.png
?      svg/custom/manually-parsed-svg-disallowed-in-dashboard-expected.checksum
?      svg/custom/manually-parsed-svg-disallowed-in-dashboard-expected.png
?      svg/custom/multiple-title-elements-expected.checksum
?      svg/custom/multiple-title-elements-expected.png
?      svg/custom/path-getTotalLength-expected.checksum
?      svg/custom/path-getTotalLength-expected.png
?      svg/custom/stroke-width-click-expected.checksum
?      svg/custom/stroke-width-click-expected.png
?      svg/custom/svg-disallowed-in-dashboard-object-expected.checksum
?      svg/custom/svg-disallowed-in-dashboard-object-expected.png
?      svg/custom/svg-features-expected.checksum
?      svg/custom/svg-features-expected.png
?      svg/custom/svgsvgelement-ctm-expected.checksum
?      svg/custom/svgsvgelement-ctm-expected.png
?      svg/custom/use-dynamic-append-expected.checksum
?      svg/dom/animated-tearoff-equality-expected.checksum
?      svg/dom/animated-tearoff-equality-expected.png
?      svg/dom/animated-tearoff-lifespan-expected.checksum
?      svg/dom/animated-tearoff-lifespan-expected.png
?      svg/dom/font-face-elements-expected.checksum
?      svg/dom/font-face-elements-expected.png
?      svg/dom/length-list-parser-expected.checksum
?      svg/dom/length-list-parser-expected.png
?      svg/dom/path-parser-expected.checksum
?      svg/dom/path-parser-expected.png
?      svg/dom/points-parser-expected.checksum
?      svg/dom/points-parser-expected.png
?      svg/dom/preserve-aspect-ratio-parser-expected.checksum
?      svg/dom/preserve-aspect-ratio-parser-expected.png
?      svg/dom/rgb-color-parser-expected.checksum
?      svg/dom/rgb-color-parser-expected.png
?      svg/dom/string-list-parser-expected.checksum
?      svg/dom/string-list-parser-expected.png
?      svg/dom/svg-element-attribute-js-null-expected.checksum
?      svg/dom/svg-element-attribute-js-null-expected.png
?      svg/dom/transform-parser-expected.checksum
?      svg/dom/transform-parser-expected.png
?      svg/hixie/dynamic/001-expected.checksum
?      svg/hixie/dynamic/001-expected.png
?      svg/webarchive/svg-cursor-subresources-expected.checksum
?      svg/webarchive/svg-cursor-subresources-expected.png
?      svg/webarchive/svg-feimage-subresources-expected.checksum
?      svg/webarchive/svg-feimage-subresources-expected.png
?      svg/webarchive/svg-script-subresouces-expected.checksum
?      svg/webarchive/svg-script-subresouces-expected.png

[Nikolas Zimmermann]

Created attachment 18316 [details]
Initial patch

[Adam Roben (:aroben)]

Comment on attachment 18316 [details]
Initial patch

r=me

[Nikolas Zimmermann]

Landed in r29235.