Bug 254792 - Avoid CPU time under AnimationEffect::getBasicTiming() when no animation event listeners are registered
Summary: Avoid CPU time under AnimationEffect::getBasicTiming() when no animation even...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Animations (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Chris Dumez
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2023-03-30 20:42 PDT by Chris Dumez
Modified: 2023-03-31 01:20 PDT (History)
3 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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>