971285 - Intermittent smil/anim-path-d-01.svg | image comparison (==), max difference: 1, number of differing pixels: 1

Status: RESOLVED FIXED

(Core :: SVG, defect)

Description

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

Attachment: Attachment 1: TEST

https://tbpl.mozilla.org/php/getParsedLog.php?id=34510083&tree=Mozilla-Inbound
slave: tegra-038



REFTEST TEST-LOAD | http://10.26.84.11:30038/tests/layout/reftests/svg/smil/anim-polyline-points-01.svg | 322 / 2354 (13%)
REFTEST TEST-LOAD | http://10.26.84.11:30038/tests/layout/reftests/svg/smil/anim-polyline-points-01-ref.svg | 322 / 2354 (13%)
REFTEST TEST-PASS | http://10.26.84.11:30038/tests/layout/reftests/svg/smil/anim-polyline-points-01.svg | image comparison (==)
REFTEST INFO | Loading a blank page
REFTEST TEST-END | http://10.26.84.11:30038/tests/layout/reftests/svg/smil/anim-polyline-points-01.svg
REFTEST TEST-START | http://10.26.84.11:30038/tests/layout/reftests/svg/smil/anim-path-d-01.svg
REFTEST TEST-LOAD | http://10.26.84.11:30038/tests/layout/reftests/svg/smil/anim-path-d-01.svg | 323 / 2354 (13%)
REFTEST TEST-LOAD | http://10.26.84.11:30038/tests/layout/reftests/svg/smil/anim-path-d-01-ref.svg | 323 / 2354 (13%)
REFTEST fuzzy match
REFTEST TEST-UNEXPECTED-FAIL | http://10.26.84.11:30038/tests/layout/reftests/svg/smil/anim-path-d-01.svg | image comparison (==), max difference: 1, number of differing pixels: 1
REFTEST   IMAGE 1 (TEST): [SEE ATTACHMENT 1 [details] [diff]]
REFTEST   IMAGE 2 (REFERENCE): [SEE ATTACHMENT 2 [details] [diff]]
REFTEST INFO | Saved log: START http://10.26.84.11:30038/tests/layout/reftests/svg/smil/anim-path-d-01.svg
REFTEST INFO | Saved log: [CONTENT] OnDocumentLoad triggering AfterOnLoadScripts
REFTEST INFO | Saved log: Initializing canvas snapshot
REFTEST INFO | Saved log: DoDrawWindow 0,0,800,1000
REFTEST INFO | Saved log: [CONTENT] RecordResult fired
REFTEST INFO | Saved log: RecordResult fired
REFTEST INFO | Saved log: START http://10.26.84.11:30038/tests/layout/reftests/svg/smil/anim-path-d-01-ref.svg
REFTEST INFO | Saved log: [CONTENT] OnDocumentLoad triggering AfterOnLoadScripts
REFTEST INFO | Saved log: Initializing canvas snapshot
REFTEST INFO | Saved log: DoDrawWindow 0,0,800,1000
REFTEST INFO | Saved log: [CONTENT] RecordResult fired
REFTEST INFO | Saved log: RecordResult fired
REFTEST INFO | Loading a blank page
REFTEST TEST-END | http://10.26.84.11:30038/tests/layout/reftests/svg/smil/anim-path-d-01.svg
REFTEST TEST-START | http://10.26.84.11:30038/tests/layout/reftests/svg/smil/anim-feComposite-operator-01.svg

Comment 1

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

Attachment: Attachment 2: Reference

Comment 2

[Legacy TBPL/Treeherder Robot]

RyanVM
https://tbpl.mozilla.org/php/getParsedLog.php?id=34670250&tree=Mozilla-Inbound
Rev4 MacOSX Snow Leopard 10.6 mozilla-inbound debug test reftest on 2014-02-13 23:15:24
revision: 226462df194e
slave: talos-r4-snow-040

REFTEST TEST-UNEXPECTED-FAIL | file:///builds/slave/talos-slave/test/build/tests/reftest/tests/layout/reftests/svg/smil/anim-path-d-01.svg | image comparison (==), max difference: 1, number of differing pixels: 18

Comment 3

[Robert Longson [:longsonr]]

Attachment: test.txt

Comment 4

[Robert Longson [:longsonr]]

The patch is supposed to be testing animation and not antialiasing so...

If this patch is ok we could make bug 726400 fixed too.

Comment 5

[Robert Longson [:longsonr]]

https://tbpl.mozilla.org/?tree=Try&rev=b13453645b39

Comment 6

[Daniel Holbert [:dholbert]]

Comment on attachment 8376342 [details] [diff] [review]
test.txt

Looks good to me!

One request:
> # animate path data:
>-skip-if(B2G) fuzzy-if(/^Windows\x20NT\x205\.1/.test(http.oscpu),2,1) fuzzy-if(cocoaWidget&&layersGPUAccelerated,1,13) == anim-path-d-01.svg anim-path-d-01-ref.svg # Bug 726400 # bug 773482
>+skip-if(B2G) == anim-path-d-01.svg anim-path-d-01-ref.svg # bug 773482

Could you also try dropping the "skip-if(B2G)" and do a Try run with B2G reftests, to see if B2G is OK on this now, too? I'll bet this might address whatever failure we had there.

r=me regardless though

Comment 7

[Robert Longson [:longsonr]]

https://tbpl.mozilla.org/?tree=Try&rev=029ed654feb3

Comment 8

[Robert Longson [:longsonr]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/072725b2ef45

Comment 9

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/mozilla-central/rev/072725b2ef45

Comment 10

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/ec87f9ef48ec
https://hg.mozilla.org/releases/mozilla-beta/rev/53cc37f62b49
https://hg.mozilla.org/releases/mozilla-b2g26_v1_2/rev/9aff0bf49198
https://hg.mozilla.org/releases/mozilla-esr24/rev/2b3e6c8ea260

Comment 11

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-b2g28_v1_3/rev/53cc37f62b49