Closed
Bug 1155651
Opened 9 years ago
Closed 9 years ago
AnimationsActor should return all animation below a given node
Categories
(DevTools :: Inspector: Animations, defect)
DevTools
Inspector: Animations
Tracking
(firefox40 fixed)
RESOLVED
FIXED
Firefox 40
Tracking | Status | |
---|---|---|
firefox40 | --- | fixed |
People
(Reporter: pbro, Assigned: pbro)
References
Details
Attachments
(1 file, 1 obsolete file)
The animation panel is more useful if when a node is selected, all animations on that node + on nodes in its subtree are displayed. This way, when <html> or <body> are selected, it's possible to see all animations. And at the same time, it's still possible to select a leaf node and see only the animations on that node if needed. This will require changes to the AnimationsActor.getAnimationPlayersForNode method. There shouldn't be any backwards compatibility problems here because the front-end has never made the assumption that all displayed players were for one node only, so it should just work if the actor method is changed.
Assignee | ||
Comment 1•9 years ago
|
||
/r/7197 - Bug 1155651 - Return all players below a node in getAnimationPlayersForNode Pull down this commit: hg pull -r 15e6b144309823031f6b345dd9bf76ccd40179f5 https://reviewboard-hg.mozilla.org/gecko/
Attachment #8593965 -
Flags: review?(mratcliffe)
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → pbrosset
Status: NEW → ASSIGNED
Assignee | ||
Comment 2•9 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=fecab36448a5
Assignee | ||
Comment 3•9 years ago
|
||
(In reply to Patrick Brosset [:pbrosset] [:patrick] from comment #2) > https://treeherder.mozilla.org/#/jobs?repo=try&revision=fecab36448a5 Of course, I didn't run the browser/devtools/animationinspector tests locally, and I should have, 3 of them are failing. That's because with this patch, selecting the <body> node in the inspector now shows all animations. It used to show no animations at all, and that's what these 3 tests were assuming. I'll fix these tests and upload a new patch for review.
Assignee | ||
Comment 4•9 years ago
|
||
Comment on attachment 8593965 [details] MozReview Request: bz://1155651/pbrosset /r/7197 - Bug 1155651 - Return all players below a node in getAnimationPlayersForNode Pull down this commit: hg pull -r f0aaf586bbd527f55dc1f47461f480b0a7ec2e43 https://reviewboard-hg.mozilla.org/gecko/
Assignee | ||
Comment 5•9 years ago
|
||
And a new try push: https://treeherder.mozilla.org/#/jobs?repo=try&revision=99e8206f9346
Assignee | ||
Comment 6•9 years ago
|
||
There are 2 remaining intermittents in the browser/devtools/animationinspector tests. Working on a fix now. This will be a test-only fix anyway, the main part of the patch won't change.
Comment on attachment 8593965 [details] MozReview Request: bz://1155651/pbrosset https://reviewboard.mozilla.org/r/7195/#review6197 Ship It!
Attachment #8593965 -
Flags: review?(mratcliffe) → review+
Assignee | ||
Comment 8•9 years ago
|
||
Thanks Mike. One last try push: https://treeherder.mozilla.org/#/jobs?repo=try&revision=a46154c80b38 All green except for seemingly unrelated failures on linux e10s. I don't see how my patch could make this test fail. So I'll push to try: https://hg.mozilla.org/integration/fx-team/rev/c419bd2f3976
https://hg.mozilla.org/mozilla-central/rev/dd8c535d665f
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
status-firefox40:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 40
Assignee | ||
Comment 11•9 years ago
|
||
Attachment #8593965 -
Attachment is obsolete: true
Attachment #8620074 -
Flags: review+
Assignee | ||
Comment 12•9 years ago
|
||
Assignee | ||
Updated•8 years ago
|
Component: Developer Tools: Inspector → Developer Tools: Animation Inspector
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•