RESOLVED FIXED 211019
[Web Animations] KeyframeEffect should ensure its target remains alive 
https://bugs.webkit.org/show_bug.cgi?id=211019
Summary [Web Animations] KeyframeEffect should ensure its target remains alive
Antoine Quint
Reported 2020-04-25 05:17:13 PDT
[Web Animations] KeyframeEffect should ensure its target remains alive
Attachments
Patch (5.13 KB, patch)
2020-04-25 05:19 PDT, Antoine Quint 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Antoine Quint
Comment 1 2020-04-25 05:19:09 PDT
Created attachment 397557 [details] Patch
Daniel Bates
Comment 2 2020-04-25 08:22:42 PDT
Comment on attachment 397557 [details] Patch Patch looks good.
EWS
Comment 3 2020-04-25 09:08:50 PDT
Committed r260705: <https://trac.webkit.org/changeset/260705> All reviewed patches have been landed. Closing bug and clearing flags on attachment 397557 [details].
Radar WebKit Bug Importer
Comment 4 2020-04-25 09:09:16 PDT
<rdar://problem/62372184>
Darin Adler
Comment 5 2020-04-25 10:26:56 PDT
Comment on attachment 397557 [details] Patch What prevents this from causing a reference cycle?
Antoine Quint
Comment 6 2020-04-26 10:47:18 PDT
(In reply to Darin Adler from comment #5) > Comment on attachment 397557 [details] > Patch > > What prevents this from causing a reference cycle? If an element is attached to the tree, document teardown will cause the effect-to-element relationship to be cleared. If the element is loose, like the one in this patch's testcase, it will potentially remain until the animation's effect is kept alive. If there are no JS references to the effect, then the procedure to remove replaced animations as defined in https://drafts.csswg.org/web-animations-1/#remove-replaced-animations will take care of this.
Note You need to log in before you can comment on or make changes to this bug.