Bug 211019

Summary: [Web Animations] KeyframeEffect should ensure its target remains alive
Product: WebKit Reporter: Antoine Quint <graouts>
Component: AnimationsAssignee: Antoine Quint <graouts>
Status: RESOLVED FIXED    
Severity: Normal CC: darin, dino, graouts, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

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.