Title: [284202] trunk/Source/WebCore
Revision
284202
Author
[email protected]
Date
2021-10-14 15:28:10 -0700 (Thu, 14 Oct 2021)

Log Message

Remove DocumentTimeline::isRunningAnimationOnRenderer()
https://bugs.webkit.org/show_bug.cgi?id=231759

Reviewed by Darin Adler.

This function has no call site.

* animation/DocumentTimeline.cpp:
(WebCore::DocumentTimeline::isRunningAnimationOnRenderer const): Deleted.
* animation/DocumentTimeline.h:

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (284201 => 284202)


--- trunk/Source/WebCore/ChangeLog	2021-10-14 22:22:05 UTC (rev 284201)
+++ trunk/Source/WebCore/ChangeLog	2021-10-14 22:28:10 UTC (rev 284202)
@@ -1,3 +1,16 @@
+2021-10-14  Antoine Quint  <[email protected]>
+
+        Remove DocumentTimeline::isRunningAnimationOnRenderer()
+        https://bugs.webkit.org/show_bug.cgi?id=231759
+
+        Reviewed by Darin Adler.
+
+        This function has no call site.
+
+        * animation/DocumentTimeline.cpp:
+        (WebCore::DocumentTimeline::isRunningAnimationOnRenderer const): Deleted.
+        * animation/DocumentTimeline.h:
+
 2021-10-14  Per Arne Vollan  <[email protected]>
 
         [AppleWin] Controls are not being rendered

Modified: trunk/Source/WebCore/animation/DocumentTimeline.cpp (284201 => 284202)


--- trunk/Source/WebCore/animation/DocumentTimeline.cpp	2021-10-14 22:22:05 UTC (rev 284201)
+++ trunk/Source/WebCore/animation/DocumentTimeline.cpp	2021-10-14 22:28:10 UTC (rev 284202)
@@ -359,28 +359,6 @@
     return true;
 }
 
-bool DocumentTimeline::isRunningAnimationOnRenderer(RenderElement& renderer, CSSPropertyID property) const
-{
-    auto styleable = Styleable::fromRenderer(renderer);
-    if (!styleable)
-        return false;
-
-    auto* animations = styleable->animations();
-    if (!animations)
-        return false;
-
-    for (const auto& animation : *animations) {
-        auto playState = animation->playState();
-        if (playState != WebAnimation::PlayState::Running && playState != WebAnimation::PlayState::Paused)
-            continue;
-        auto* effect = animation->effect();
-        if (is<KeyframeEffect>(effect) && downcast<KeyframeEffect>(effect)->animatedProperties().contains(property))
-            return true;
-    }
-
-    return false;
-}
-
 bool DocumentTimeline::isRunningAcceleratedAnimationOnRenderer(RenderElement& renderer, CSSPropertyID property) const
 {
     auto styleable = Styleable::fromRenderer(renderer);

Modified: trunk/Source/WebCore/animation/DocumentTimeline.h (284201 => 284202)


--- trunk/Source/WebCore/animation/DocumentTimeline.h	2021-10-14 22:22:05 UTC (rev 284201)
+++ trunk/Source/WebCore/animation/DocumentTimeline.h	2021-10-14 22:28:10 UTC (rev 284202)
@@ -60,7 +60,6 @@
     // transform animation but we were unable to cheaply compute its effect on the extent.
     bool computeExtentOfAnimation(RenderElement&, LayoutRect&) const;
     std::unique_ptr<RenderStyle> animatedStyleForRenderer(RenderElement& renderer);
-    bool isRunningAnimationOnRenderer(RenderElement&, CSSPropertyID) const;
     bool isRunningAcceleratedAnimationOnRenderer(RenderElement&, CSSPropertyID) const;
     void animationAcceleratedRunningStateDidChange(WebAnimation&);
     bool runningAnimationsForRendererAreAllAccelerated(const RenderBoxModelObject&) const;
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to