Bug 1289049 Part 4 - Make <basic-shape> animatable for shape-outside for gecko style system. r=boris

MozReview-Commit-ID: 7x0jdqWqkpL

--HG--
extra : rebase_source : e79a0ca24fb929f80d305adf164329ae0f44690f

layout/style/StyleAnimationValue.cpp

@@ -4585,6 +4585,16 @@ StyleAnimationValue::ExtractComputedValue(nsCSSPropertyID aProperty,
           break;
         }
 
+        case eCSSProperty_shape_outside: {
+          const nsStyleDisplay* styleDisplay =
+            static_cast<const nsStyleDisplay*>(styleStruct);
+          if (!ExtractComputedValueFromShapeSource(styleDisplay->mShapeOutside,
+                                                   aComputedValue)) {
+            return false;
+          };
+          break;
+        }
+
         case eCSSProperty_filter: {
           const nsStyleEffects* effects =
             static_cast<const nsStyleEffects*>(styleStruct);

layout/style/nsCSSPropList.h

@@ -3757,7 +3757,7 @@ CSS_PROP_DISPLAY(
     0,
     nullptr,
     CSS_PROP_NO_OFFSET,
-    eStyleAnimType_Discrete) // FIXME: Bug 1289049 for adding animation support
+    eStyleAnimType_Custom)
 CSS_PROP_SVG(
     shape-rendering,
     shape_rendering,

layout/style/test/test_transitions_per_property.html

@@ -286,9 +286,7 @@ var supported_properties = {
                                    test_true_currentcolor_transition ]
 };
 
-// Bug 1289049: After making shape-outside animatable on Gecko, we can remove
-// |isServo| flag from this if-condition.
-if (SpecialPowers.getBoolPref("layout.css.shape-outside.enabled") && isServo) {
+if (SpecialPowers.getBoolPref("layout.css.shape-outside.enabled")) {
   supported_properties["shape-outside"] =
     [ test_basic_shape_or_url_transition ];
 }