Intermittent svg/smil/anim-gradient-attr-presence-01.svg == svg/smil/anim-gradient-attr-presence-01.svg | image comparison, max difference: 1, number of differing pixels: 4736

RESOLVED FIXED in Firefox 55

Status

()

defect
P4
normal
RESOLVED FIXED
2 years ago
8 months ago

People

(Reporter: intermittent-bug-filer, Assigned: hiro)

Tracking

({intermittent-failure})

unspecified
mozilla55
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox55 fixed)

Details

(Whiteboard: [stockwell fixed:other])

Attachments

(1 attachment)

Priority: -- → P4
Comment hidden (Intermittent Failures Robot)
Comment hidden (Intermittent Failures Robot)
Comment hidden (Intermittent Failures Robot)
Comment hidden (Intermittent Failures Robot)
This test is already fuzzy-if'd at https://dxr.mozilla.org/mozilla-central/rev/cad53f061da634a16ea75887558301b77f65745d/layout/reftests/svg/smil/reftest.list#241 - including stylo might be an easy win here.
Whiteboard: [stockwell needswork]
(In reply to Geoff Brown [:gbrown] from comment #5)
> This test is already fuzzy-if'd at
> https://dxr.mozilla.org/mozilla-central/rev/
> cad53f061da634a16ea75887558301b77f65745d/layout/reftests/svg/smil/reftest.
> list#241 - including stylo might be an easy win here.

Actually, this failure has a much greater # of different pixels -- all the edges.
:hiro - This started failing shortly after bug 1357295 -- related?
Flags: needinfo?(hikezoe)
Assignee

Comment 8

2 years ago
Yes, of course, this is related to bug 1367327, but the failure frequency raised after bug 1367327?  It's really odd, since bug 1367327 changed stroke-width parser to match to gecko's behavior.
This started failing on May 14, shortly after bug 1357295 landed, and increased in frequency May 28, a couple of days after bug 1367327 landed (but that was a weekend, and the increase was "only" about 2x, so, its hard to say if the increase was related to 1367327).
this week we are on track for the same failure rate as last week.

:hiro, can you pick up fixing this bug?  either random-if or skip-if is ok, possibly there is a real fix we can do.
Comment hidden (mozreview-request)

Comment 13

2 years ago
mozreview-review
Comment on attachment 8876530 [details]
Bug 1364731 - Specify px unit explicitly.

https://reviewboard.mozilla.org/r/147856/#review152198

I'm fine with doing this in the interests of enabling this test, but I also feel like we may as well just fix the underlying issue (precision of lengths in transform lists) and not touch this test.
Attachment #8876530 - Flags: review?(bbirtles) → review+
(In reply to Brian Birtles (:birtles) from comment #13)
> Comment on attachment 8876530 [details]
> Bug 1364731 - Specify px unit explicitly.
> 
> https://reviewboard.mozilla.org/r/147856/#review152198
> 
> I'm fine with doing this in the interests of enabling this test,

Actually, now I see this is an intermittent failure. In that case this is totally fine!
Comment hidden (Intermittent Failures Robot)

Comment 16

2 years ago
Pushed by hikezoe@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/3118b9ee9db8
Specify px unit explicitly. r=birtles
Assignee

Comment 17

2 years ago
Leaving ni to me here since I should provide a test case which reliably fails due to the precision mismatch.

Comment 18

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/3118b9ee9db8
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
Assignee: nobody → hikezoe
Whiteboard: [stockwell needswork] → [stockwell fixed:other]
Flags: needinfo?(hikezoe)
You need to log in before you can comment on or make changes to this bug.