LibWeb: Replace ListStyleStyleValue with ShorthandStyleValue

Author: AtkinsSJ

Meta/gn/secondary/Userland/Libraries/LibWeb/CSS/StyleValues/BUILD.gn

@@ -24,7 +24,6 @@ source_set("StyleValues") {
     "IntegerStyleValue.cpp",
     "LengthStyleValue.cpp",
     "LinearGradientStyleValue.cpp",
-    "ListStyleStyleValue.cpp",
     "MathDepthStyleValue.cpp",
     "NumberStyleValue.cpp",
     "OverflowStyleValue.cpp",

Userland/Libraries/LibWeb/CMakeLists.txt

@@ -101,7 +101,6 @@ set(SOURCES
     CSS/StyleValues/IntegerStyleValue.cpp
     CSS/StyleValues/LengthStyleValue.cpp
     CSS/StyleValues/LinearGradientStyleValue.cpp
-    CSS/StyleValues/ListStyleStyleValue.cpp
     CSS/StyleValues/MathDepthStyleValue.cpp
     CSS/StyleValues/NumberStyleValue.cpp
     CSS/StyleValues/OverflowStyleValue.cpp

Userland/Libraries/LibWeb/CSS/Parser/Parser.cpp

@@ -57,7 +57,6 @@
 #include <LibWeb/CSS/StyleValues/InitialStyleValue.h>
 #include <LibWeb/CSS/StyleValues/IntegerStyleValue.h>
 #include <LibWeb/CSS/StyleValues/LengthStyleValue.h>
-#include <LibWeb/CSS/StyleValues/ListStyleStyleValue.h>
 #include <LibWeb/CSS/StyleValues/MathDepthStyleValue.h>
 #include <LibWeb/CSS/StyleValues/NumberStyleValue.h>
 #include <LibWeb/CSS/StyleValues/OverflowStyleValue.h>
@@ -4498,7 +4497,9 @@ RefPtr<StyleValue> Parser::parse_list_style_value(Vector<ComponentValue> const&
     if (!list_type)
         list_type = property_initial_value(m_context.realm(), PropertyID::ListStyleType);
 
-    return ListStyleStyleValue::create(list_position.release_nonnull(), list_image.release_nonnull(), list_type.release_nonnull());
+    return ShorthandStyleValue::create(PropertyID::ListStyle,
+        { PropertyID::ListStylePosition, PropertyID::ListStyleImage, PropertyID::ListStyleType },
+        { list_position.release_nonnull(), list_image.release_nonnull(), list_type.release_nonnull() });
 }
 
 RefPtr<StyleValue> Parser::parse_math_depth_value(Vector<ComponentValue> const& component_values)

Userland/Libraries/LibWeb/CSS/StyleComputer.cpp

@@ -42,7 +42,6 @@
 #include <LibWeb/CSS/StyleValues/IdentifierStyleValue.h>
 #include <LibWeb/CSS/StyleValues/IntegerStyleValue.h>
 #include <LibWeb/CSS/StyleValues/LengthStyleValue.h>
-#include <LibWeb/CSS/StyleValues/ListStyleStyleValue.h>
 #include <LibWeb/CSS/StyleValues/MathDepthStyleValue.h>
 #include <LibWeb/CSS/StyleValues/NumberStyleValue.h>
 #include <LibWeb/CSS/StyleValues/OverflowStyleValue.h>
@@ -705,14 +704,6 @@ static void set_property_expanding_shorthands(StyleProperties& style, CSS::Prope
     }
 
     if (property_id == CSS::PropertyID::ListStyle) {
-        if (value.is_list_style()) {
-            auto const& list_style = value.as_list_style();
-            set_longhand_property(CSS::PropertyID::ListStylePosition, list_style.position());
-            set_longhand_property(CSS::PropertyID::ListStyleImage, list_style.image());
-            set_longhand_property(CSS::PropertyID::ListStyleType, list_style.style_type());
-            return;
-        }
-
         set_longhand_property(CSS::PropertyID::ListStylePosition, value);
         set_longhand_property(CSS::PropertyID::ListStyleImage, value);
         set_longhand_property(CSS::PropertyID::ListStyleType, value);

Userland/Libraries/LibWeb/CSS/StyleValue.cpp

@@ -37,7 +37,6 @@
 #include <LibWeb/CSS/StyleValues/IntegerStyleValue.h>
 #include <LibWeb/CSS/StyleValues/LengthStyleValue.h>
 #include <LibWeb/CSS/StyleValues/LinearGradientStyleValue.h>
-#include <LibWeb/CSS/StyleValues/ListStyleStyleValue.h>
 #include <LibWeb/CSS/StyleValues/MathDepthStyleValue.h>
 #include <LibWeb/CSS/StyleValues/NumberStyleValue.h>
 #include <LibWeb/CSS/StyleValues/OverflowStyleValue.h>

Userland/Libraries/LibWeb/CSS/StyleValue.h

@@ -109,7 +109,6 @@ using StyleValueVector = Vector<ValueComparingNonnullRefPtr<StyleValue const>>;
     __ENUMERATE_STYLE_VALUE_TYPE(Integer, integer)                                            \
     __ENUMERATE_STYLE_VALUE_TYPE(Length, length)                                              \
     __ENUMERATE_STYLE_VALUE_TYPE(LinearGradient, linear_gradient)                             \
-    __ENUMERATE_STYLE_VALUE_TYPE(ListStyle, list_style)                                       \
     __ENUMERATE_STYLE_VALUE_TYPE(MathDepth, math_depth)                                       \
     __ENUMERATE_STYLE_VALUE_TYPE(Number, number)                                              \
     __ENUMERATE_STYLE_VALUE_TYPE(Overflow, overflow)                                          \

Userland/Libraries/LibWeb/CSS/StyleValues/ListStyleStyleValue.cpp

@@ -145,6 +145,8 @@ String ShorthandStyleValue::to_string() const
             return MUST(String::formatted("{}", construct_rows_string()));
         return MUST(String::formatted("{} / {}", construct_rows_string(), columns.grid_track_size_list().to_string()));
     }
+    case PropertyID::ListStyle:
+        return MUST(String::formatted("{} {} {}", longhand(PropertyID::ListStylePosition)->to_string(), longhand(PropertyID::ListStyleImage)->to_string(), longhand(PropertyID::ListStyleType)->to_string()));
     default:
         StringBuilder builder;
         auto first = true;

Userland/Libraries/LibWeb/CSS/StyleValues/ListStyleStyleValue.h

@@ -127,7 +127,6 @@ class LengthOrCalculated;
 class LengthPercentage;
 class LengthStyleValue;
 class LinearGradientStyleValue;
-class ListStyleStyleValue;
 class MathDepthStyleValue;
 class MediaFeatureValue;
 class MediaList;