Bug 1383650 - Add SVG geometry property support in CSS r=emilio
☠☠ backed out by 1624c5a31917 ☠ ☠
authorviolet <violet.bugreport@gmail.com>
Thu, 16 May 2019 00:47:20 +0000
changeset 532844 70e74dd6b45d7c14074263f5a47631783aa89780
parent 532843 55a283e793dfd42add9d085f2b32bee82a226946
child 532845 18b030b3166027a1a1e3d35ed779bdc4d7c6ab85
push id11272
push userapavel@mozilla.com
push dateThu, 16 May 2019 15:28:22 +0000
treeherdermozilla-beta@2265bfc5920d [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersemilio
bugs1383650
milestone68.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1383650 - Add SVG geometry property support in CSS r=emilio This patch adds SVG geometry properties to CSS, it doesn't deal with how SVG handles them. Differential Revision: https://phabricator.services.mozilla.com/D29937
layout/style/nsStyleStruct.cpp
layout/style/nsStyleStruct.h
layout/style/test/property_database.js
layout/style/test/test_transitions_per_property.html
servo/components/style/properties/longhands/svg.mako.rs
testing/web-platform/meta/svg/extensibility/foreignObject/properties.svg.ini
testing/web-platform/meta/svg/geometry/parsing/cx-valid.svg.ini
testing/web-platform/meta/svg/geometry/parsing/cy-valid.svg.ini
testing/web-platform/meta/svg/geometry/parsing/height-computed.svg.ini
testing/web-platform/meta/svg/geometry/parsing/r-valid.svg.ini
testing/web-platform/meta/svg/geometry/parsing/rx-valid.svg.ini
testing/web-platform/meta/svg/geometry/parsing/ry-valid.svg.ini
testing/web-platform/meta/svg/geometry/parsing/width-computed.svg.ini
testing/web-platform/meta/svg/geometry/parsing/x-valid.svg.ini
testing/web-platform/meta/svg/geometry/parsing/y-valid.svg.ini
testing/web-platform/meta/svg/shapes/rx-ry-not-inherited.svg.ini
--- a/layout/style/nsStyleStruct.cpp
+++ b/layout/style/nsStyleStruct.cpp
@@ -1064,32 +1064,46 @@ void nsStyleFilter::SetDropShadow(nsCSSS
   mDropShadow->AddRef();
   mType = NS_STYLE_FILTER_DROP_SHADOW;
 }
 
 // --------------------
 // nsStyleSVGReset
 //
 nsStyleSVGReset::nsStyleSVGReset(const Document& aDocument)
-    : mMask(nsStyleImageLayers::LayerType::Mask),
+    : mX(LengthPercentage::Zero()),
+      mY(LengthPercentage::Zero()),
+      mCx(LengthPercentage::Zero()),
+      mCy(LengthPercentage::Zero()),
+      mRx(NonNegativeLengthPercentageOrAuto::Auto()),
+      mRy(NonNegativeLengthPercentageOrAuto::Auto()),
+      mR(NonNegativeLengthPercentage::Zero()),
+      mMask(nsStyleImageLayers::LayerType::Mask),
       mStopColor(StyleColor::Black()),
       mFloodColor(StyleColor::Black()),
       mLightingColor(StyleColor::White()),
       mStopOpacity(1.0f),
       mFloodOpacity(1.0f),
       mDominantBaseline(NS_STYLE_DOMINANT_BASELINE_AUTO),
       mVectorEffect(NS_STYLE_VECTOR_EFFECT_NONE),
       mMaskType(NS_STYLE_MASK_TYPE_LUMINANCE) {
   MOZ_COUNT_CTOR(nsStyleSVGReset);
 }
 
 nsStyleSVGReset::~nsStyleSVGReset() { MOZ_COUNT_DTOR(nsStyleSVGReset); }
 
 nsStyleSVGReset::nsStyleSVGReset(const nsStyleSVGReset& aSource)
-    : mMask(aSource.mMask),
+    : mX(aSource.mX),
+      mY(aSource.mY),
+      mCx(aSource.mCx),
+      mCy(aSource.mCy),
+      mRx(aSource.mRx),
+      mRy(aSource.mRy),
+      mR(aSource.mR),
+      mMask(aSource.mMask),
       mClipPath(aSource.mClipPath),
       mStopColor(aSource.mStopColor),
       mFloodColor(aSource.mFloodColor),
       mLightingColor(aSource.mLightingColor),
       mStopOpacity(aSource.mStopOpacity),
       mFloodOpacity(aSource.mFloodOpacity),
       mDominantBaseline(aSource.mDominantBaseline),
       mVectorEffect(aSource.mVectorEffect),
@@ -1131,16 +1145,22 @@ void nsStyleSVGReset::TriggerImageLoads(
     }
   }
 }
 
 nsChangeHint nsStyleSVGReset::CalcDifference(
     const nsStyleSVGReset& aNewData) const {
   nsChangeHint hint = nsChangeHint(0);
 
+  if (mX != aNewData.mX || mY != aNewData.mY || mCx != aNewData.mCx ||
+      mCy != aNewData.mCy || mR != aNewData.mR || mRx != aNewData.mRx ||
+      mRy != aNewData.mRy) {
+    hint |= nsChangeHint_InvalidateRenderingObservers | nsChangeHint_NeedReflow;
+  }
+
   if (mClipPath != aNewData.mClipPath) {
     hint |= nsChangeHint_UpdateEffects | nsChangeHint_RepaintFrame;
   }
 
   if (mDominantBaseline != aNewData.mDominantBaseline) {
     // XXXjwatt: why NS_STYLE_HINT_REFLOW? Isn't that excessive?
     hint |= NS_STYLE_HINT_REFLOW;
   } else if (mVectorEffect != aNewData.mVectorEffect) {
--- a/layout/style/nsStyleStruct.h
+++ b/layout/style/nsStyleStruct.h
@@ -2854,16 +2854,25 @@ struct MOZ_NEEDS_MEMMOVABLE_MEMBERS nsSt
   }
 
   bool HasMask() const;
 
   bool HasNonScalingStroke() const {
     return mVectorEffect == NS_STYLE_VECTOR_EFFECT_NON_SCALING_STROKE;
   }
 
+  // geometry properties
+  mozilla::LengthPercentage mX;
+  mozilla::LengthPercentage mY;
+  mozilla::LengthPercentage mCx;
+  mozilla::LengthPercentage mCy;
+  mozilla::NonNegativeLengthPercentageOrAuto mRx;
+  mozilla::NonNegativeLengthPercentageOrAuto mRy;
+  mozilla::NonNegativeLengthPercentage mR;
+
   nsStyleImageLayers mMask;
   mozilla::StyleShapeSource mClipPath;
   mozilla::StyleColor mStopColor;
   mozilla::StyleColor mFloodColor;
   mozilla::StyleColor mLightingColor;
 
   float mStopOpacity;
   float mFloodOpacity;
--- a/layout/style/test/property_database.js
+++ b/layout/style/test/property_database.js
@@ -5627,16 +5627,72 @@ var gCSSProperties = {
   "stroke-width": {
     domProp: "strokeWidth",
     inherited: true,
     type: CSS_TYPE_LONGHAND,
     initial_values: [ "1px" ],
     other_values: [ "0", "0px", "-0em", "17px", "0.2em", "0.0002", "context-value" ],
     invalid_values: [ "-0.1px", "-3px" ]
   },
+  "x": {
+    domProp: "x",
+    inherited: false,
+    type: CSS_TYPE_LONGHAND,
+    initial_values: [ "0px" ],
+    other_values: [ "-1em", "17px", "0.2em", "23.4%" ],
+    invalid_values: [ "auto", "context-value", "0.0002" ]
+  },
+  "y": {
+    domProp: "y",
+    inherited: false,
+    type: CSS_TYPE_LONGHAND,
+    initial_values: [ "0px" ],
+    other_values: [ "-1em", "17px", "0.2em", "23.4%" ],
+    invalid_values: [ "auto", "context-value", "0.0002" ]
+  },
+  "cx": {
+    domProp: "cx",
+    inherited: false,
+    type: CSS_TYPE_LONGHAND,
+    initial_values: [ "0px" ],
+    other_values: [ "-1em", "17px", "0.2em", "23.4%" ],
+    invalid_values: [ "auto", "context-value", "0.0002" ]
+  },
+  "cy": {
+    domProp: "cy",
+    inherited: false,
+    type: CSS_TYPE_LONGHAND,
+    initial_values: [ "0px" ],
+    other_values: [ "-1em", "17px", "0.2em", "23.4%" ],
+    invalid_values: [ "auto", "context-value", "0.0002" ]
+  },
+  "r": {
+    domProp: "r",
+    inherited: false,
+    type: CSS_TYPE_LONGHAND,
+    initial_values: [ "0px" ],
+    other_values: [ "17px", "0.2em", "23.4%" ],
+    invalid_values: [ "auto", "-1", "-1.5px", "0.0002" ]
+  },
+  "rx": {
+    domProp: "rx",
+    inherited: false,
+    type: CSS_TYPE_LONGHAND,
+    initial_values: [ "auto" ],
+    other_values: [ "17px", "0.2em", "23.4%" ],
+    invalid_values: [ "hello", "-12px", "0.0002" ]
+  },
+  "ry": {
+    domProp: "ry",
+    inherited: false,
+    type: CSS_TYPE_LONGHAND,
+    initial_values: [ "auto" ],
+    other_values: [ "17px", "0.2em", "23.4%" ],
+    invalid_values: [ "hello", "-1.3px", "0.0002" ]
+  },
   "text-anchor": {
     domProp: "textAnchor",
     inherited: true,
     type: CSS_TYPE_LONGHAND,
     initial_values: [ "start" ],
     other_values: [ "middle", "end" ],
     invalid_values: []
   },
--- a/layout/style/test/test_transitions_per_property.html
+++ b/layout/style/test/test_transitions_per_property.html
@@ -75,16 +75,20 @@ var supported_properties = {
     "column-count": [ test_pos_integer_or_auto_transition,
                       test_integer_at_least_one_clamping ],
     "column-rule-color": [ test_color_transition,
                            test_currentcolor_transition ],
     "column-rule-width": [ test_length_transition,
                            test_length_clamped ],
     "column-width": [ test_length_transition,
                       test_length_clamped ],
+    "cx": [ test_length_transition, test_percent_transition,
+            test_length_unclamped, test_percent_unclamped ],
+    "cy": [ test_length_transition, test_percent_transition,
+            test_length_unclamped, test_percent_unclamped ],
     "-moz-image-region": [ test_rect_transition ],
     "-moz-outline-radius-bottomleft": [ test_radius_transition ],
     "-moz-outline-radius-bottomright": [ test_radius_transition ],
     "-moz-outline-radius-topleft": [ test_radius_transition ],
     "-moz-outline-radius-topright": [ test_radius_transition ],
     "background-color": [ test_color_transition,
                           test_currentcolor_transition ],
     "background-position": [ test_background_position_transition,
@@ -239,16 +243,22 @@ var supported_properties = {
                      test_length_clamped, test_percent_clamped ],
     "perspective": [ test_length_transition ],
     "perspective-origin": [ test_length_pair_transition,
                             test_length_percent_pair_transition,
                             test_length_percent_pair_unclamped ],
     "right": [ test_length_transition, test_percent_transition,
                test_length_percent_calc_transition,
                test_length_unclamped, test_percent_unclamped ],
+    "r": [ test_length_transition, test_percent_transition,
+           test_length_clamped, test_percent_clamped ],
+    "rx": [ test_length_transition, test_percent_transition,
+            test_length_clamped, test_percent_clamped ],
+    "ry": [ test_length_transition, test_percent_transition,
+            test_length_clamped, test_percent_clamped ],
     "shape-image-threshold": [ test_float_zeroToOne_transition,
                                // shape-image-threshold (like opacity) is
                                // clamped in computed style
                                // (not parsing/interpolation)
                                test_float_zeroToOne_clamped ],
     "shape-margin": [ test_length_transition, test_percent_transition,
                       test_length_clamped, test_percent_clamped ],
     "shape-outside": [ test_basic_shape_or_url_transition ],
@@ -292,16 +302,20 @@ var supported_properties = {
                           test_length_percent_pair_unclamped ],
     "vertical-align": [ test_length_transition, test_percent_transition,
                         test_length_unclamped, test_percent_unclamped ],
     "visibility": [ test_visibility_transition ],
     "width": [ test_length_transition, test_percent_transition,
                test_length_percent_calc_transition,
                test_length_clamped, test_percent_clamped ],
     "word-spacing": [ test_length_transition, test_length_unclamped ],
+    "x": [ test_length_transition, test_percent_transition,
+           test_length_unclamped, test_percent_unclamped ],
+    "y": [ test_length_transition, test_percent_transition,
+           test_length_unclamped, test_percent_unclamped ],
     "z-index": [ test_integer_transition, test_pos_integer_or_auto_transition ],
     "-webkit-line-clamp": [ test_pos_integer_or_none_transition ],
     "-webkit-text-fill-color": [ test_color_transition,
                                  test_currentcolor_transition ],
     "-webkit-text-stroke-color": [ test_color_transition,
                                    test_currentcolor_transition ]
 };
 
--- a/servo/components/style/properties/longhands/svg.mako.rs
+++ b/servo/components/style/properties/longhands/svg.mako.rs
@@ -186,8 +186,71 @@
     parse_method="parse_with_cors_anonymous",
     spec="https://drafts.fxtf.org/css-masking/#propdef-mask-image",
     vector=True,
     products="gecko",
     extra_prefixes="webkit",
     animation_value_type="discrete",
     flags="CREATES_STACKING_CONTEXT",
 )}
+
+${helpers.predefined_type(
+    "x",
+    "LengthPercentage",
+    "computed::LengthPercentage::zero()",
+    products="gecko",
+    animation_value_type="ComputedValue",
+    spec="https://svgwg.org/svg2-draft/geometry.html#X",
+)}
+
+${helpers.predefined_type(
+    "y",
+    "LengthPercentage",
+    "computed::LengthPercentage::zero()",
+    products="gecko",
+    animation_value_type="ComputedValue",
+    spec="https://svgwg.org/svg2-draft/geometry.html#Y",
+)}
+
+${helpers.predefined_type(
+    "cx",
+    "LengthPercentage",
+    "computed::LengthPercentage::zero()",
+    products="gecko",
+    animation_value_type="ComputedValue",
+    spec="https://svgwg.org/svg2-draft/geometry.html#CX",
+)}
+
+${helpers.predefined_type(
+    "cy",
+    "LengthPercentage",
+    "computed::LengthPercentage::zero()",
+    products="gecko",
+    animation_value_type="ComputedValue",
+    spec="https://svgwg.org/svg2-draft/geometry.html#CY",
+)}
+
+${helpers.predefined_type(
+    "rx",
+    "NonNegativeLengthPercentageOrAuto",
+    "computed::NonNegativeLengthPercentageOrAuto::auto()",
+    products="gecko",
+    animation_value_type="LengthPercentageOrAuto",
+    spec="https://svgwg.org/svg2-draft/geometry.html#RX",
+)}
+
+${helpers.predefined_type(
+    "ry",
+    "NonNegativeLengthPercentageOrAuto",
+    "computed::NonNegativeLengthPercentageOrAuto::auto()",
+    products="gecko",
+    animation_value_type="LengthPercentageOrAuto",
+    spec="https://svgwg.org/svg2-draft/geometry.html#RY",
+)}
+
+${helpers.predefined_type(
+    "r",
+    "NonNegativeLengthPercentage",
+    "computed::NonNegativeLengthPercentage::zero()",
+    products="gecko",
+    animation_value_type="LengthPercentage",
+    spec="https://svgwg.org/svg2-draft/geometry.html#R",
+)}
--- a/testing/web-platform/meta/svg/extensibility/foreignObject/properties.svg.ini
+++ b/testing/web-platform/meta/svg/extensibility/foreignObject/properties.svg.ini
@@ -1,19 +1,6 @@
 [properties.svg]
-  [Untitled]
-    expected: FAIL
-
   [width and height default to auto]
     expected: FAIL
 
-  [style rules are applied]
-    expected: FAIL
-
-  [attributes set properties]
-    expected: FAIL
-
-  [style rules override attributes]
-    expected: FAIL
-
   [width and height default to auto (which computes to "0px")]
     expected: FAIL
-
deleted file mode 100644
--- a/testing/web-platform/meta/svg/geometry/parsing/cx-valid.svg.ini
+++ /dev/null
@@ -1,16 +0,0 @@
-[cx-valid.svg]
-  [e.style['cx'\] = "4%" should set the property value]
-    expected: FAIL
-
-  [e.style['cx'\] = "-1px" should set the property value]
-    expected: FAIL
-
-  [e.style['cx'\] = "calc(2em + 3ex)" should set the property value]
-    expected: FAIL
-
-  [e.style['cx'\] = "0" should set the property value]
-    expected: FAIL
-
-  [e.style['cx'\] = "5ch" should set the property value]
-    expected: FAIL
-
deleted file mode 100644
--- a/testing/web-platform/meta/svg/geometry/parsing/cy-valid.svg.ini
+++ /dev/null
@@ -1,16 +0,0 @@
-[cy-valid.svg]
-  [e.style['cy'\] = "0" should set the property value]
-    expected: FAIL
-
-  [e.style['cy'\] = "calc(2em + 3ex)" should set the property value]
-    expected: FAIL
-
-  [e.style['cy'\] = "4%" should set the property value]
-    expected: FAIL
-
-  [e.style['cy'\] = "-1px" should set the property value]
-    expected: FAIL
-
-  [e.style['cy'\] = "5rem" should set the property value]
-    expected: FAIL
-
--- a/testing/web-platform/meta/svg/geometry/parsing/height-computed.svg.ini
+++ b/testing/web-platform/meta/svg/geometry/parsing/height-computed.svg.ini
@@ -1,24 +1,18 @@
 [height-computed.svg]
   [SVG Geometry Properties: getComputedStyle().height, <svg> inline style (percentage)]
     expected: FAIL
 
   [SVG Geometry Properties: getComputedStyle().height, <rect> initial]
     expected: FAIL
 
-  [SVG Geometry Properties: getComputedStyle().height, <rect> presentation attribute]
-    expected: FAIL
-
   [SVG Geometry Properties: getComputedStyle().height, <rect> inline style (auto)]
     expected: FAIL
 
-  [SVG Geometry Properties: getComputedStyle().height, <foreignObject> presentation attribute]
-    expected: FAIL
-
   [SVG Geometry Properties: getComputedStyle().height, <image> initial]
     expected: FAIL
 
   [SVG Geometry Properties: getComputedStyle().height, <svg> inline style (auto)]
     expected: FAIL
 
   [SVG Geometry Properties: getComputedStyle().height, <foreignObject> initial]
     expected: FAIL
@@ -39,11 +33,8 @@
     expected: FAIL
 
   [SVG Geometry Properties: getComputedStyle().height, <foreignObject> inline style (percentage)]
     expected: FAIL
 
   [SVG Geometry Properties: getComputedStyle().height, <image> inline style (percentage)]
     expected: FAIL
 
-  [SVG Geometry Properties: getComputedStyle().height, <image> presentation attribute]
-    expected: FAIL
-
deleted file mode 100644
--- a/testing/web-platform/meta/svg/geometry/parsing/r-valid.svg.ini
+++ /dev/null
@@ -1,16 +0,0 @@
-[r-valid.svg]
-  [e.style['r'\] = "1px" should set the property value]
-    expected: FAIL
-
-  [e.style['r'\] = "0" should set the property value]
-    expected: FAIL
-
-  [e.style['r'\] = "calc(2em + 3ex)" should set the property value]
-    expected: FAIL
-
-  [e.style['r'\] = "4%" should set the property value]
-    expected: FAIL
-
-  [e.style['r'\] = "5vmin" should set the property value]
-    expected: FAIL
-
deleted file mode 100644
--- a/testing/web-platform/meta/svg/geometry/parsing/rx-valid.svg.ini
+++ /dev/null
@@ -1,19 +0,0 @@
-[rx-valid.svg]
-  [e.style['rx'\] = "auto" should set the property value]
-    expected: FAIL
-
-  [e.style['rx'\] = "4%" should set the property value]
-    expected: FAIL
-
-  [e.style['rx'\] = "calc(2em + 3ex)" should set the property value]
-    expected: FAIL
-
-  [e.style['rx'\] = "0" should set the property value]
-    expected: FAIL
-
-  [e.style['rx'\] = "1px" should set the property value]
-    expected: FAIL
-
-  [e.style['rx'\] = "5vw" should set the property value]
-    expected: FAIL
-
deleted file mode 100644
--- a/testing/web-platform/meta/svg/geometry/parsing/ry-valid.svg.ini
+++ /dev/null
@@ -1,37 +0,0 @@
-[ry-valid.svg]
-  [e.style['rx'\] = "auto" should set the property value]
-    expected: FAIL
-
-  [e.style['rx'\] = "4%" should set the property value]
-    expected: FAIL
-
-  [e.style['rx'\] = "calc(2em + 3ex)" should set the property value]
-    expected: FAIL
-
-  [e.style['rx'\] = "0" should set the property value]
-    expected: FAIL
-
-  [e.style['rx'\] = "1px" should set the property value]
-    expected: FAIL
-
-  [e.style['rx'\] = "5vh" should set the property value]
-    expected: FAIL
-
-  [e.style['ry'\] = "auto" should set the property value]
-    expected: FAIL
-
-  [e.style['ry'\] = "0" should set the property value]
-    expected: FAIL
-
-  [e.style['ry'\] = "calc(2em + 3ex)" should set the property value]
-    expected: FAIL
-
-  [e.style['ry'\] = "4%" should set the property value]
-    expected: FAIL
-
-  [e.style['ry'\] = "1px" should set the property value]
-    expected: FAIL
-
-  [e.style['ry'\] = "5vh" should set the property value]
-    expected: FAIL
-
--- a/testing/web-platform/meta/svg/geometry/parsing/width-computed.svg.ini
+++ b/testing/web-platform/meta/svg/geometry/parsing/width-computed.svg.ini
@@ -9,28 +9,22 @@
     expected: FAIL
 
   [SVG Geometry Properties: getComputedStyle().width, <svg> inline style (auto)]
     expected: FAIL
 
   [SVG Geometry Properties: getComputedStyle().width, <rect> initial]
     expected: FAIL
 
-  [SVG Geometry Properties: getComputedStyle().width, <rect> presentation attribute]
-    expected: FAIL
-
   [SVG Geometry Properties: getComputedStyle().width, <image> inline style (percentage)]
     expected: FAIL
 
   [SVG Geometry Properties: getComputedStyle().width, <svg> presentation attribute]
     expected: FAIL
 
-  [SVG Geometry Properties: getComputedStyle().width, <foreignObject> presentation attribute]
-    expected: FAIL
-
   [SVG Geometry Properties: getComputedStyle().width, <foreignObject> inline style (percentage)]
     expected: FAIL
 
   [SVG Geometry Properties: getComputedStyle().width, <rect> inline style (auto)]
     expected: FAIL
 
   [SVG Geometry Properties: getComputedStyle().width, <foreignObject> initial]
     expected: FAIL
@@ -39,11 +33,8 @@
     expected: FAIL
 
   [SVG Geometry Properties: getComputedStyle().width, <svg> initial]
     expected: FAIL
 
   [SVG Geometry Properties: getComputedStyle().width, <image> inline style (auto)]
     expected: FAIL
 
-  [SVG Geometry Properties: getComputedStyle().width, <image> presentation attribute]
-    expected: FAIL
-
deleted file mode 100644
--- a/testing/web-platform/meta/svg/geometry/parsing/x-valid.svg.ini
+++ /dev/null
@@ -1,16 +0,0 @@
-[x-valid.svg]
-  [e.style['x'\] = "-1px" should set the property value]
-    expected: FAIL
-
-  [e.style['x'\] = "0" should set the property value]
-    expected: FAIL
-
-  [e.style['x'\] = "calc(2em + 3ex)" should set the property value]
-    expected: FAIL
-
-  [e.style['x'\] = "4%" should set the property value]
-    expected: FAIL
-
-  [e.style['x'\] = "5cm" should set the property value]
-    expected: FAIL
-
deleted file mode 100644
--- a/testing/web-platform/meta/svg/geometry/parsing/y-valid.svg.ini
+++ /dev/null
@@ -1,16 +0,0 @@
-[y-valid.svg]
-  [e.style['y'\] = "-1px" should set the property value]
-    expected: FAIL
-
-  [e.style['y'\] = "4%" should set the property value]
-    expected: FAIL
-
-  [e.style['y'\] = "0" should set the property value]
-    expected: FAIL
-
-  [e.style['y'\] = "calc(2em + 3ex)" should set the property value]
-    expected: FAIL
-
-  [e.style['y'\] = "5mm" should set the property value]
-    expected: FAIL
-
deleted file mode 100644
--- a/testing/web-platform/meta/svg/shapes/rx-ry-not-inherited.svg.ini
+++ /dev/null
@@ -1,7 +0,0 @@
-[rx-ry-not-inherited.svg]
-  [Untitled]
-    expected: FAIL
-
-  [rx-ry-not-inherited]
-    expected: FAIL
-