Make font-size-adjust animatable.

Source/core/animation/EffectInput.cpp

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 25 matching lines @@
 #include "core/animation/AnimationInputHelpers.h"
 #include "core/animation/KeyframeEffectModel.h"
 #include "core/animation/StringKeyframe.h"
 #include "core/css/resolver/StyleResolver.h"
 #include "core/dom/Document.h"
 #include "core/dom/Element.h"
 #include "wtf/NonCopyingSort.h"
 
 namespace blink {
 
+static bool isValidValueForInterpolation(const CSSPropertyID& property, const CSSValue& value)
+{
+    const CSSPrimitiveValue& primitiveValue = toCSSPrimitiveValue(value);
+
+    switch (property) {
+    case CSSPropertyFontSizeAdjust:
+        return primitiveValue.getDoubleValue(CSSPrimitiveValue::CSS_NUMBER) > 0;
+    default:
+        break;
+    }
+    return true;
+}
+
 PassRefPtrWillBeRawPtr<AnimationEffect> EffectInput::convert(Element* element, const Vector<Dictionary>& keyframeDictionaryVector, ExceptionState& exceptionState)
 {
     // FIXME: Remove the dependency on element.
     if (!element)
         return nullptr;
 
     StyleSheetContents* styleSheetContents = element->document().elementSheet().contents();
     StringKeyframeVector keyframes;
+    PropertySet invalidValueProperties;
     double lastOffset = 0;
 
     for (const auto& keyframeDictionary : keyframeDictionaryVector) {
         RefPtrWillBeRawPtr<StringKeyframe> keyframe = StringKeyframe::create();
 
         ScriptValue scriptValue;
         bool frameHasOffset = DictionaryHelper::get(keyframeDictionary, "offset", scriptValue) && !scriptValue.isNull();
 
         if (frameHasOffset) {
             double offset;
@@ skipping 33 matching lines @@
 
         Vector<String> keyframeProperties;
         keyframeDictionary.getPropertyNames(keyframeProperties);
         for (const auto& property : keyframeProperties) {
             CSSPropertyID id = AnimationInputHelpers::keyframeAttributeToCSSPropertyID(property);
             if (id == CSSPropertyInvalid)
                 continue;
             String value;
             DictionaryHelper::get(keyframeDictionary, property, value);
             keyframe->setPropertyValue(id, value, styleSheetContents);
+
+            if (!isValidValueForInterpolation(id, *keyframe->propertyValue(id)))
+                invalidValueProperties.add(id);
         }
     }
 
+    for (CSSPropertyID property : invalidValueProperties) {
+        for (const auto& keyframe : keyframes)
+            keyframe->clearPropertyValue(property);
+    }
+
     RefPtrWillBeRawPtr<StringKeyframeEffectModel> keyframeEffectModel = StringKeyframeEffectModel::create(keyframes);
     if (keyframeEffectModel->hasSyntheticKeyframes()) {
         exceptionState.throwDOMException(NotSupportedError, "Partial keyframes are not supported.");
         return nullptr;
     }
     if (!keyframeEffectModel->isReplaceOnly()) {
         exceptionState.throwDOMException(NotSupportedError, "Additive animations are not supported.");
         return nullptr;
     }
     keyframeEffectModel->forceConversionsToAnimatableValues(*element);
 
     return keyframeEffectModel;
 }
 
 PassRefPtrWillBeRawPtr<AnimationEffect> EffectInput::convert(Element* element, const AnimationEffectOrDictionarySequence& effectInput, ExceptionState& exceptionState)
 {
     if (effectInput.isAnimationEffect())
         return effectInput.getAsAnimationEffect();
     if (effectInput.isDictionarySequence())
         return convert(element, effectInput.getAsDictionarySequence(), exceptionState);
     return nullptr;
 }
 
 } // namespace blink