Bug 1338087 - Part 1: Don't apply paced spacing on stylo.
To avoid using StyleAnimationValue on stylo, we should skip
ApplySpacing, which means We always fall back to distribute spacing on stylo.
MozReview-Commit-ID: 9DNwhOI8saf
--- a/dom/animation/KeyframeEffectReadOnly.cpp
+++ b/dom/animation/KeyframeEffectReadOnly.cpp
@@ -891,17 +891,20 @@ KeyframeEffectReadOnly::BuildProperties(
// make a copy of |mKeyframes| first and iterate over that instead.
auto keyframesCopy(mKeyframes);
nsTArray<ComputedKeyframeValues> computedValues =
KeyframeUtils::GetComputedKeyframeValues(keyframesCopy,
mTarget->mElement,
aStyleContext);
- if (mEffectOptions.mSpacingMode == SpacingMode::paced) {
+ // FIXME: Bug 1332633: we have to implement ComputeDistance for
+ // RawServoAnimationValue.
+ if (mEffectOptions.mSpacingMode == SpacingMode::paced &&
+ aStyleContext->PresContext()->StyleSet()->IsGecko()) {
KeyframeUtils::ApplySpacing(keyframesCopy, SpacingMode::paced,
mEffectOptions.mPacedProperty,
computedValues, aStyleContext);
}
result =
KeyframeUtils::GetAnimationPropertiesFromKeyframes(keyframesCopy,
computedValues,