[webkit-changes] [WebKit/WebKit] 2700ab: Avoid CPU time under AnimationEffect::getBasicTimi...

[Chris Dumez]

  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 2700abd1cd65088c94c06be6e3d711dbee1561a5
      https://github.com/WebKit/WebKit/commit/2700abd1cd65088c94c06be6e3d711dbee1561a5
  Author: Chris Dumez <cdumez at apple.com>
  Date:   2023-03-31 (Fri, 31 Mar 2023)

  Changed paths:
    M Source/WebCore/animation/DeclarativeAnimation.cpp
    M Source/WebCore/animation/DeclarativeAnimation.h

  Log Message:
  -----------
  Avoid CPU time under AnimationEffect::getBasicTiming() when no animation event listeners are registered
https://bugs.webkit.org/show_bug.cgi?id=254792

Reviewed by Antoine Quint.

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)

* Source/WebCore/animation/DeclarativeAnimation.cpp:
(WebCore::DeclarativeAnimation::tick):
(WebCore::DeclarativeAnimation::cancel):
(WebCore::DeclarativeAnimation::shouldFireDOMEvents const):
(WebCore::DeclarativeAnimation::invalidateDOMEvents):
* Source/WebCore/animation/DeclarativeAnimation.h:

Canonical link: https://commits.webkit.org/262390@main