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

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

Issue 1690543002: Add additive animation support for CSS properties perspective-origin and object-position (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@_backgroundPositionAnimation
Patch Set: Rebased Created 4 years, 10 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/CSSPositionAxisListInterpolationType.cpp
diff --git a/third_party/WebKit/Source/core/animation/CSSPositionAxisListInterpolationType.cpp b/third_party/WebKit/Source/core/animation/CSSPositionAxisListInterpolationType.cpp
index 93d79823aaaa3af4b6158a7f75e8120fd7e84fad..2aeb6d294c31527ba17de6d0d1428d1afec6a6a2 100644
--- a/third_party/WebKit/Source/core/animation/CSSPositionAxisListInterpolationType.cpp
+++ b/third_party/WebKit/Source/core/animation/CSSPositionAxisListInterpolationType.cpp
@@ -12,17 +12,34 @@
namespace blink {
-static InterpolationValue convertPositionAxisCSSValue(const CSSValue& value)
+InterpolationValue CSSPositionAxisListInterpolationType::convertPositionAxisCSSValue(const CSSValue& value)
{
- if (!value.isValuePair())
+ if (value.isValuePair()) {
+ const CSSValuePair& pair = toCSSValuePair(value);
+ InterpolationValue result = CSSLengthInterpolationType::maybeConvertCSSValue(pair.second());
+ CSSValueID side = toCSSPrimitiveValue(pair.first()).getValueID();
+ if (side == CSSValueRight || side == CSSValueBottom)
+ CSSLengthInterpolationType::subtractFromOneHundredPercent(result);
+ return result;
+ }
+
+ const CSSPrimitiveValue& primitveValue = toCSSPrimitiveValue(value);
+ if (!primitveValue.isValueID())
return CSSLengthInterpolationType::maybeConvertCSSValue(value);
- const CSSValuePair& pair = toCSSValuePair(value);
- InterpolationValue result = CSSLengthInterpolationType::maybeConvertCSSValue(pair.second());
- CSSValueID side = toCSSPrimitiveValue(pair.first()).getValueID();
- if (side == CSSValueRight || side == CSSValueBottom)
- CSSLengthInterpolationType::subtractFromOneHundredPercent(result);
- return result;
+ switch (primitveValue.getValueID()) {
+ case CSSValueLeft:
+ case CSSValueTop:
+ return CSSLengthInterpolationType::createInterpolablePercent(0);
+ case CSSValueRight:
+ case CSSValueBottom:
+ return CSSLengthInterpolationType::createInterpolablePercent(100);
+ case CSSValueCenter:
+ return CSSLengthInterpolationType::createInterpolablePercent(50);
+ default:
+ ASSERT_NOT_REACHED();
+ return nullptr;
+ }
}
InterpolationValue CSSPositionAxisListInterpolationType::maybeConvertValue(const CSSValue& value, const StyleResolverState&, ConversionCheckers&) const

Powered by Google App Engine
This is Rietveld 408576698