Chromium Code Reviews| Index: Source/core/animation/AnimationStack.cpp |
| diff --git a/Source/core/animation/AnimationStack.cpp b/Source/core/animation/AnimationStack.cpp |
| index d9645baebc7937b8c5962bd4550cdbd773676993..0a022c4c8605a5acd8ea7e6c52b066990dd8d3f9 100644 |
| --- a/Source/core/animation/AnimationStack.cpp |
| +++ b/Source/core/animation/AnimationStack.cpp |
| @@ -73,8 +73,9 @@ AnimationStack::AnimationStack() |
| bool AnimationStack::hasActiveAnimationsOnCompositor(CSSPropertyID property) const |
| { |
| - for (const auto& effect : m_effects) { |
| - if (effect->effect() && effect->effect()->hasActiveAnimationsOnCompositor(property)) |
| + for (const auto& sampledEffect : m_effects) { |
| + // TODO(dstockwell): move the playing check into AnimationEffect and expose both hasAnimations and hasActiveAnimations |
| + if (sampledEffect->effect() && sampledEffect->effect()->animation()->playing() && sampledEffect->effect()->hasActiveAnimationsOnCompositor(property)) |
|
esprehn
2015/08/26 08:44:28
Can you explain why this fixes the bug? Your descr
|
| return true; |
| } |
| return false; |