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

Description Chris Dumez 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)

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

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

Comment 2 EWS 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.

Comment 3 Radar WebKit Bug Importer 2023-03-31 01:20:17 PDT

<rdar://problem/107461572>