Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(234)

Unified Diff: Source/core/animation/ActiveAnimations.cpp

Issue 189883003: Web Animations: Allow animations with delay to start on compositor (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: FIXME, renaming. Created 6 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: Source/core/animation/ActiveAnimations.cpp
diff --git a/Source/core/animation/ActiveAnimations.cpp b/Source/core/animation/ActiveAnimations.cpp
index 317722af295e12a085355f33bfe410d3fefb7cdf..0f530a0e8de3f096dc5734d57530da6d34dde770 100644
--- a/Source/core/animation/ActiveAnimations.cpp
+++ b/Source/core/animation/ActiveAnimations.cpp
@@ -77,7 +77,16 @@ bool hasActiveAnimationsOnCompositor(const RenderObject& renderer, CSSPropertyID
bool ActiveAnimations::hasActiveAnimations(CSSPropertyID property) const
{
- return m_defaultStack.affects(property);
+ for (PlayerSet::const_iterator it = m_players.begin(); it != players().end(); ++it) {
+ const Player& player = *it->key;
+ ASSERT(player.source());
+ // FIXME: Needs to consider AnimationGroup once added.
+ ASSERT(player.source()->isAnimation());
+ const Animation& animation = *toAnimation(player.source());
+ if (animation.isCurrent() && animation.affects(property))
+ return true;
+ }
+ return false;
}
bool ActiveAnimations::hasActiveAnimationsOnCompositor(CSSPropertyID property) const

Powered by Google App Engine
This is Rietveld 408576698