[webkit-reviews] review granted: [Bug 228833] REGRESSION (r268932): CPU usage higher than expected with sibling elements running WebAnimations : [Attachment 443227] Patch

[bugzilla-daemon at webkit.org]

Antti Koivisto <koivisto at iki.fi> has granted Antoine Quint
<graouts at webkit.org>'s request for review:
Bug 228833: REGRESSION (r268932): CPU usage higher than expected with sibling
elements running WebAnimations
https://bugs.webkit.org/show_bug.cgi?id=228833

Attachment 443227: Patch

https://bugs.webkit.org/attachment.cgi?id=443227&action=review




--- Comment #13 from Antti Koivisto <koivisto at iki.fi> ---
Comment on attachment 443227
  --> https://bugs.webkit.org/attachment.cgi?id=443227
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=443227&action=review

> Source/WebCore/style/StyleSharingResolver.cpp:263
> -    if (style->transitions() || style->animations())
> +    if (candidateElement.hasKeyframeEffects(PseudoId::None))

Transitions are "keyframe effects" too?