Changeset 290571 in webkit

Timestamp:

Feb 27, 2022 2:15:50 PM (5 months ago)

Author:

graouts@webkit.org

Message:

[svg] flood-opacity should be clamped to the [0,1] range
​https://bugs.webkit.org/show_bug.cgi?id=237253

Reviewed by Dean Jackson.

LayoutTests/imported/w3c:

• web-platform-tests/web-animations/animation-model/animation-types/accumulation-per-property-001-expected.txt:
• web-platform-tests/web-animations/animation-model/animation-types/addition-per-property-001-expected.txt:

Source/WebCore:

• rendering/style/SVGRenderStyle.h:

(WebCore::SVGRenderStyle::setFloodOpacity):

Location:

trunk

Files:

• LayoutTests/imported/w3c/ChangeLog (modified) (1 diff)
• LayoutTests/imported/w3c/web-platform-tests/web-animations/animation-model/animation-types/accumulation-per-property-001-expected.txt (modified) (1 diff)
• LayoutTests/imported/w3c/web-platform-tests/web-animations/animation-model/animation-types/addition-per-property-001-expected.txt (modified) (1 diff)
• Source/WebCore/ChangeLog (modified) (1 diff)
• Source/WebCore/rendering/style/SVGRenderStyle.h (modified) (1 diff)

trunk/LayoutTests/imported/w3c/ChangeLog

@@ +1 @@
+2022-02-27  Antoine Quint  <graouts@webkit.org>
+
+        [svg] flood-opacity should be clamped to the [0,1] range
+        https://bugs.webkit.org/show_bug.cgi?id=237253
+
+        Reviewed by Dean Jackson.
+
+        * web-platform-tests/web-animations/animation-model/animation-types/accumulation-per-property-001-expected.txt:
+        * web-platform-tests/web-animations/animation-model/animation-types/addition-per-property-001-expected.txt:
+
 2022-02-27  Antoine Quint  <graouts@webkit.org>
 

trunk/LayoutTests/imported/w3c/web-platform-tests/web-animations/animation-model/animation-types/accumulation-per-property-001-expected.txt

@@ -210 +210 @@
 PASS flood-opacity (type: opacity) has testAccumulation function
 PASS flood-opacity: [0, 1] number
-FAIL flood-opacity: [0, 1] number (clamped) assert_equals: The value should be 1 at 0ms expected "1" but got "1.1"
+PASS flood-opacity: [0, 1] number (clamped)
 PASS font-stretch (type: percentage) has testAccumulation function
 PASS font-stretch: percentage

trunk/LayoutTests/imported/w3c/web-platform-tests/web-animations/animation-model/animation-types/addition-per-property-001-expected.txt

@@ -210 +210 @@
 PASS flood-opacity (type: opacity) has testAddition function
 PASS flood-opacity: [0, 1] number
-FAIL flood-opacity: [0, 1] number (clamped) assert_equals: The value should be 1 at 0ms expected "1" but got "1.1"
+PASS flood-opacity: [0, 1] number (clamped)
 PASS font-stretch (type: percentage) has testAddition function
 PASS font-stretch: percentage

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2022-02-27  Antoine Quint  <graouts@webkit.org>
+
+        [svg] flood-opacity should be clamped to the [0,1] range
+        https://bugs.webkit.org/show_bug.cgi?id=237253
+
+        Reviewed by Dean Jackson.
+
+        * rendering/style/SVGRenderStyle.h:
+        (WebCore::SVGRenderStyle::setFloodOpacity):
+
 2022-02-27  Antoine Quint  <graouts@webkit.org>
 

trunk/Source/WebCore/rendering/style/SVGRenderStyle.h

@@ -368 +368 @@
 inline void SVGRenderStyle::setFloodOpacity(float opacity)
 {
-    if (!(m_miscData->floodOpacity == opacity))
-        m_miscData.access().floodOpacity = opacity;
+    auto clampedOpacity = clampTo<float>(opacity, 0.f, 1.f);
+    if (!(m_miscData->floodOpacity == clampedOpacity))
+        m_miscData.access().floodOpacity = clampedOpacity;
 }