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

Unified Diff: third_party/WebKit/Source/core/animation/KeyframeEffect.cpp

Issue 2241993002: CSS Motion Path: New names for properties (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: review comments Created 4 years, 3 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: third_party/WebKit/Source/core/animation/KeyframeEffect.cpp
diff --git a/third_party/WebKit/Source/core/animation/KeyframeEffect.cpp b/third_party/WebKit/Source/core/animation/KeyframeEffect.cpp
index e0e76491fb6446edc3317a145a4c2313d8f1366c..a7ed19229003fcc0b8c818c9f086410859e760aa 100644
--- a/third_party/WebKit/Source/core/animation/KeyframeEffect.cpp
+++ b/third_party/WebKit/Source/core/animation/KeyframeEffect.cpp
@@ -162,7 +162,7 @@ bool KeyframeEffect::hasIncompatibleStyle()
|| animation()->affects(*m_target, CSSPropertyTranslate);
if (animation()->hasActiveAnimationsOnCompositor()) {
- if (m_target->computedStyle()->hasMotionPath() && affectsTransform)
+ if (m_target->computedStyle()->hasOffsetPath() && affectsTransform)
return true;
return hasMultipleTransformProperties();
}
@@ -281,7 +281,7 @@ bool KeyframeEffect::isCandidateForAnimationOnCompositor(double animationPlaybac
// in computed style because they need to be explicitly ordered
if (!model()
|| !m_target
- || (m_target->computedStyle() && m_target->computedStyle()->hasMotionPath())
+ || (m_target->computedStyle() && m_target->computedStyle()->hasOffsetPath())
|| hasMultipleTransformProperties())
return false;

Powered by Google App Engine
This is Rietveld 408576698