[SVG2] Change default value of fx and fy to 50% for SVGRadialGradient

Ahmad-S792 · Ahmad Saleem · commit 42a0c2981cc0 · 2026-02-04T14:18:46.000-08:00
https://bugs.webkit.org/show_bug.cgi?id=306976 rdar://169645572 Reviewed by Said Abou-Hallawa. This patch aligns with Web Specification [1]: As per web specification change, the initial value for 'fx' and 'fy' is now '50%' and this change follows it. [1] w3c/svgwg@762c875 * Source/WebCore/svg/SVGRadialGradientElement.h: * LayoutTests/imported/w3c/web-platform-tests/svg/types/scripted/SVGAnimatedLength-initial-values-expected.txt: Progressions Canonical link: https://commits.webkit.org/306811@main
diff --git a/LayoutTests/imported/w3c/web-platform-tests/svg/types/scripted/SVGAnimatedLength-initial-values-expected.txt b/LayoutTests/imported/w3c/web-platform-tests/svg/types/scripted/SVGAnimatedLength-initial-values-expected.txt
@@ -91,10 +91,10 @@ PASS SVGAnimatedLength, initial values, SVGRadialGradientElement.prototype.cy (r
 PASS SVGAnimatedLength, initial values, SVGRadialGradientElement.prototype.cy (invalid value)
 PASS SVGAnimatedLength, initial values, SVGRadialGradientElement.prototype.r (remove)
 PASS SVGAnimatedLength, initial values, SVGRadialGradientElement.prototype.r (invalid value)
-FAIL SVGAnimatedLength, initial values, SVGRadialGradientElement.prototype.fx (remove) assert_equals: initial before expected "50%" but got "0"
-FAIL SVGAnimatedLength, initial values, SVGRadialGradientElement.prototype.fx (invalid value) assert_equals: initial before expected "50%" but got "0"
-FAIL SVGAnimatedLength, initial values, SVGRadialGradientElement.prototype.fy (remove) assert_equals: initial before expected "50%" but got "0"
-FAIL SVGAnimatedLength, initial values, SVGRadialGradientElement.prototype.fy (invalid value) assert_equals: initial before expected "50%" but got "0"
+PASS SVGAnimatedLength, initial values, SVGRadialGradientElement.prototype.fx (remove)
+PASS SVGAnimatedLength, initial values, SVGRadialGradientElement.prototype.fx (invalid value)
+PASS SVGAnimatedLength, initial values, SVGRadialGradientElement.prototype.fy (remove)
+PASS SVGAnimatedLength, initial values, SVGRadialGradientElement.prototype.fy (invalid value)
 PASS SVGAnimatedLength, initial values, SVGRadialGradientElement.prototype.fr (remove)
 PASS SVGAnimatedLength, initial values, SVGRadialGradientElement.prototype.fr (invalid value)
 PASS SVGAnimatedLength, initial values, SVGRectElement.prototype.x (remove)
diff --git a/Source/WebCore/svg/SVGRadialGradientElement.h b/Source/WebCore/svg/SVGRadialGradientElement.h
@@ -67,8 +67,8 @@ class SVGRadialGradientElement final : public SVGGradientElement {
     Ref<SVGAnimatedLength> m_cx { SVGAnimatedLength::create(this, SVGLengthMode::Width, "50%"_s) };
     Ref<SVGAnimatedLength> m_cy { SVGAnimatedLength::create(this, SVGLengthMode::Height, "50%"_s) };
     Ref<SVGAnimatedLength> m_r { SVGAnimatedLength::create(this, SVGLengthMode::Other, "50%"_s) };
-    Ref<SVGAnimatedLength> m_fx { SVGAnimatedLength::create(this, SVGLengthMode::Width) };
-    Ref<SVGAnimatedLength> m_fy { SVGAnimatedLength::create(this, SVGLengthMode::Height) };
+    Ref<SVGAnimatedLength> m_fx { SVGAnimatedLength::create(this, SVGLengthMode::Width, "50%"_s) };
+    Ref<SVGAnimatedLength> m_fy { SVGAnimatedLength::create(this, SVGLengthMode::Height, "50%"_s) };
     Ref<SVGAnimatedLength> m_fr { SVGAnimatedLength::create(this, SVGLengthMode::Other, "0%"_s) };
 };
 
