Bug 254792

Summary:	Avoid CPU time under AnimationEffect::getBasicTiming() when no animation event listeners are registered
Product:	WebKit	Reporter:	Chris Dumez <cdumez>
Component:	Animations	Assignee:	Chris Dumez <cdumez>
Status:	RESOLVED FIXED
Severity:	Normal	CC:	dino, graouts, webkit-bug-importer
Priority:	P2	Keywords:	InRadar
Version:	WebKit Nightly Build
Hardware:	Unspecified
OS:	Unspecified

Chris Dumez

Reported 2023-03-30 20:42:55 PDT

Avoid CPU time under AnimationEffect::getBasicTiming() when no animation event listeners are registered: Sample Count, Samples %, Normalized CPU %, Symbol 45, 0.0%, 0.0%, WebCore::AnimationEffect::getBasicTiming(std::__1::optional<WTF::Seconds>) const (in WebCore) 25, 0.0%, 0.0%, WebCore::DeclarativeAnimation::cancelFromStyle() (in WebCore) 8, 0.0%, 0.0%, WebCore::WebAnimation::updateRelevance() (in WebCore) 5, 0.0%, 0.0%, WebCore::Styleable::cancelDeclarativeAnimations() const (in WebCore)

Attachments
Add attachment proposed patch, testcase, etc.

Chris Dumez

Comment 1 2023-03-30 20:44:29 PDT

Pull request: https://github.com/WebKit/WebKit/pull/12223

EWS

Comment 2 2023-03-31 01:19:39 PDT

Committed 262390@main (2700abd1cd65): <https://commits.webkit.org/262390@main> Reviewed commits have been landed. Closing PR #12223 and removing active labels.

Radar WebKit Bug Importer

Comment 3 2023-03-31 01:20:17 PDT

<rdar://problem/107461572>

Note You need to log in before you can comment on or make changes to this bug.