Closed
Bug 1432061
Opened 8 years ago
Closed 8 years ago
Let nsTransitionManager participate in cycle collection
Categories
(Core :: DOM: Animation, enhancement)
Tracking
()
RESOLVED
FIXED
mozilla60
| Tracking | Status | |
|---|---|---|
| firefox60 | --- | fixed |
People
(Reporter: hiro, Assigned: hiro)
Details
Attachments
(1 file)
We did it for nsAnimationManager [1], but we didn't it for nsTransitionManager.
https://treeherder.mozilla.org/#/jobs?repo=try&revision=ef0bd61d59c55a0bca402e2f9010bf8b78e64fd3
[1] https://hg.mozilla.org/mozilla-central/rev/7c94690a852b
| Comment hidden (mozreview-request) |
|
||
Comment 2•8 years ago
|
||
| mozreview-review | ||
Comment on attachment 8944309 [details]
Bug 1432061 - Let nsTransformManager participate in cycle-collection.
https://reviewboard.mozilla.org/r/214540/#review220130
I feel like there was a reason for not doing this previously (perhaps due to the way transition events were dispatched--prior to moving the dispatch to the start of the refresh tick) but I think it makes sense to do this now. Thanks for noticing this.
Attachment #8944309 -
Flags: review?(bbirtles) → review+
Pushed by hikezoe@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/2c17ccaca1a2
Let nsTransformManager participate in cycle-collection. r=birtles
|
||
Comment 4•8 years ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
status-firefox60:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla60
You need to log in
before you can comment on or make changes to this bug.
Description
•