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

Unified Diff: sky/engine/core/animation/DocumentAnimations.cpp

Issue 772673002: Fix Animations, Remove Compostior Animations. (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: CompositorPendingAnimations -> PendingAnimations Created 6 years 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
« no previous file with comments | « sky/engine/core/animation/DocumentAnimations.h ('k') | sky/engine/core/animation/PendingAnimations.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sky/engine/core/animation/DocumentAnimations.cpp
diff --git a/sky/engine/core/animation/DocumentAnimations.cpp b/sky/engine/core/animation/DocumentAnimations.cpp
index a3699a136555162040e68682c453214f3cc4f12c..ea622c8bd6613eb6174cc42e6bfa7774535adbce 100644
--- a/sky/engine/core/animation/DocumentAnimations.cpp
+++ b/sky/engine/core/animation/DocumentAnimations.cpp
@@ -64,20 +64,6 @@ void DocumentAnimations::updateOutdatedAnimationPlayersIfNeeded(Document& docume
updateAnimationTiming(document, TimingUpdateOnDemand);
}
-void DocumentAnimations::updateAnimationTimingForGetComputedStyle(Node& node, CSSPropertyID property)
-{
- if (!node.isElementNode())
- return;
- const Element& element = toElement(node);
- if (RenderStyle* style = element.renderStyle()) {
- if ((property == CSSPropertyOpacity && style->isRunningOpacityAnimationOnCompositor())
- || ((property == CSSPropertyTransform || property == CSSPropertyWebkitTransform) && style->isRunningTransformAnimationOnCompositor())
- || (property == CSSPropertyWebkitFilter && style->isRunningFilterAnimationOnCompositor())) {
- updateAnimationTiming(element.document(), TimingUpdateOnDemand);
- }
- }
-}
-
bool DocumentAnimations::needsOutdatedAnimationPlayerUpdate(const Document& document)
{
return document.timeline().hasOutdatedAnimationPlayer();
@@ -87,7 +73,7 @@ bool DocumentAnimations::needsOutdatedAnimationPlayerUpdate(const Document& docu
void DocumentAnimations::startPendingAnimations(Document& document)
{
ASSERT(document.lifecycle().state() == DocumentLifecycle::PaintInvalidationClean);
- if (document.compositorPendingAnimations().update()) {
+ if (document.pendingAnimations().update()) {
ASSERT(document.view());
document.view()->scheduleAnimation();
}
« no previous file with comments | « sky/engine/core/animation/DocumentAnimations.h ('k') | sky/engine/core/animation/PendingAnimations.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698