Bug 1383650 - Add SVG geometry property support in CSS r=emilio
authorviolet <violet.bugreport@gmail.com>
Thu, 16 May 2019 13:21:19 +0000
changeset 474698 ae39b084c0b1e7c6ca2fb07af2d3c52678a44405
parent 474697 d2660d957f4212c0f80ed2c65487cabd300988a4
child 474699 b3152421e832cf876396bb7e7c1ad874cb174c90
push id36044
push userrmaries@mozilla.com
push dateTue, 21 May 2019 15:45:34 +0000
treeherdermozilla-central@78571bb1f20e [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersemilio
bugs1383650
milestone69.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
@@ -1056,32 +1056,46 @@ void nsStyleFilter::SetDropShadow(const 
   new (&mDropShadow) StyleSimpleShadow(aSrc);
   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),
@@ -1123,16 +1137,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
@@ -2698,16 +2698,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
@@ -5642,16 +5642,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
-