Closed
Bug 1261651
Opened 9 years ago
Closed 9 years ago
Various animation inspector tests are going to permafail when Gecko 48 merges to Aurora (TypeError: el.animate is not a function at @http://example.com/browser/devtools/client/animationinspector/test/doc_simple_animation.html:120:5 )
Categories
(DevTools :: Inspector: Animations, defect, P1)
DevTools
Inspector: Animations
Tracking
(firefox46 unaffected, firefox47+ fixed, firefox48+ fixed)
RESOLVED
FIXED
Firefox 48
Tracking | Status | |
---|---|---|
firefox46 | --- | unaffected |
firefox47 | + | fixed |
firefox48 | + | fixed |
People
(Reporter: RyanVM, Assigned: pbro)
References
Details
(Whiteboard: [btpp-fix-now])
Attachments
(1 file)
[Tracking Requested - why for this release]: Devtools permafail when Gecko 48 merges to Aurora in two weeks.
https://treeherder.mozilla.org/logviewer.html#?job_id=18975270&repo=try
TEST-UNEXPECTED-FAIL | devtools/client/animationinspector/test/browser_animation_click_selects_animation.js | uncaught exception - TypeError: el.animate is not a function at @http://example.com/browser/devtools/client/animationinspector/test/doc_simple_animation.html:120:5
Flags: needinfo?(motoryo1)
Reporter | ||
Comment 1•9 years ago
|
||
Patrick, maybe you can take a look? We're a bit short on time here given where we are in the release cycle :)
Flags: needinfo?(pbrosset)
Assignee | ||
Comment 2•9 years ago
|
||
el.animate is the main piece of the WebAnimations API that is being worked on at the moment. It almost shipped in FF47, but was pushed back to FF48. Which means that for the time being it is being disabled on purpose after nightly. So it seems expected that these tests will fail when merged in Aurora. Most probably, the fix to turn the feature back on on other non-nightly releases hasn't been done yet.
See bug 1253507.
Brian Birtles would know a lot more than me on this, but he's away until April 10 and not accepting NI?
NI? Boris who reviewed the patch, so he can confirm my understanding.
Flags: needinfo?(pbrosset)
Flags: needinfo?(motoryo1)
Flags: needinfo?(bzbarsky)
Comment 3•9 years ago
|
||
The feature is not enabled on non-nightly, correct.
So the tests need to either be disabled on non-nightly (with a bug filed to reenable them once the feature is enabled) or enable the feature for the duration of the tests.
Flags: needinfo?(bzbarsky)
Reporter | ||
Comment 4•9 years ago
|
||
I'm actually in the process of adding nightly_build to mozinfo, so the former should be an option in the near-future (with a fail-if condition so the tests UNEXPECTED-PASS when the feature is ready to ride the trains), but in general I think the latter (preffing it on in the test) would be preferable.
Assignee | ||
Comment 5•9 years ago
|
||
Triaging as P1. I'll take a look at this bug in the coming days, making sure the tests set the right pref before starting.
Assignee: nobody → pbrosset
Status: NEW → ASSIGNED
Priority: -- → P1
Whiteboard: [btpp-fix-now]
Assignee | ||
Comment 6•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/44963/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/44963/
Attachment #8738949 -
Flags: review?(mratcliffe)
Assignee | ||
Comment 7•9 years ago
|
||
I ended up taking a third approach: using KeyframeEffect and Animation constructors to create animations rather than element.animate.
Comment on attachment 8738949 [details]
MozReview Request: Bug 1261651 - Avoid using element.animate() in tests - Use Animation constructor isntead; r=miker
https://reviewboard.mozilla.org/r/44963/#review41515
Attachment #8738949 -
Flags: review?(mratcliffe) → review+
Assignee | ||
Comment 9•9 years ago
|
||
Comment 11•9 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 48
tracking-firefox47:
--- → +
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•