LibWeb: Store stop-opacity in computed form in ComputedProperties

Author: https://github.com/Calme1709
Commit: c045969234
Pull-request: https://github.com/LadybirdBrowser/ladybird/pull/6112
Reviewed-by: https://github.com/AtkinsSJ
Reviewed-by: https://github.com/tcl3 ✅

Libraries/LibWeb/CSS/CSSStyleProperties.cpp

@@ -813,10 +813,6 @@ RefPtr<StyleValue const> CSSStyleProperties::style_value_for_computed_property(L
 
         return get_computed_value(property_id);
     }
-    case PropertyID::StopOpacity: {
-        auto opacity = layout_node.computed_values().stop_opacity();
-        return NumberStyleValue::create(opacity);
-    }
     case PropertyID::StrokeOpacity: {
         auto opacity = layout_node.computed_values().stroke_opacity();
         return NumberStyleValue::create(opacity);

Libraries/LibWeb/CSS/ComputedProperties.cpp

@@ -515,8 +515,7 @@ float ComputedProperties::stroke_opacity() const
 
 float ComputedProperties::stop_opacity() const
 {
-    auto const& value = property(PropertyID::StopOpacity);
+    return property(PropertyID::StopOpacity).as_number().number();
-    return resolve_opacity_value(value);
 }
 
 FillRule ComputedProperties::fill_rule() const

Libraries/LibWeb/CSS/StyleComputer.cpp

@@ -3285,6 +3285,7 @@ NonnullRefPtr<StyleValue const> StyleComputer::compute_value_of_property(Propert
         return compute_border_or_outline_width(specified_value, get_property_specified_value(PropertyID::OutlineStyle), computation_context);
     case PropertyID::FillOpacity:
     case PropertyID::Opacity:
+    case PropertyID::StopOpacity:
         return compute_opacity(specified_value, computation_context);
     default:
         // FIXME: We should replace this with a VERIFY_NOT_REACHED() once all properties have their own handling.