From 8d83dce1d7fc897a06642c0a9be861795e14c3c5 Mon Sep 17 00:00:00 2001
From: Aliaksandr Kalenik <kalenik.aliaksandr@gmail.com>
Date: Fri, 13 Jun 2025 17:12:04 +0200
Subject: [PATCH] LibWeb: Use variant to represent CSS::ExplicitGridTrack

No behavior change.
---
 Libraries/LibWeb/CSS/GridTrackSize.cpp        | 38 ++---------
 Libraries/LibWeb/CSS/GridTrackSize.h          | 64 ++++---------------
 .../LibWeb/Layout/GridFormattingContext.cpp   | 11 +---
 3 files changed, 19 insertions(+), 94 deletions(-)

diff --git a/Libraries/LibWeb/CSS/GridTrackSize.cpp b/Libraries/LibWeb/CSS/GridTrackSize.cpp
index 3e4f26f5b32..33f2dfa4637 100644
--- a/Libraries/LibWeb/CSS/GridTrackSize.cpp
+++ b/Libraries/LibWeb/CSS/GridTrackSize.cpp
@@ -179,44 +179,16 @@ String GridRepeat::to_string() const
     return MUST(builder.to_string());
 }
 
-ExplicitGridTrack::ExplicitGridTrack(CSS::GridFitContent grid_fit_content)
-    : m_type(Type::FitContent)
-    , m_grid_fit_content(grid_fit_content)
-{
-}
-
-ExplicitGridTrack::ExplicitGridTrack(CSS::GridMinMax grid_minmax)
-    : m_type(Type::MinMax)
-    , m_grid_minmax(grid_minmax)
-{
-}
-
-ExplicitGridTrack::ExplicitGridTrack(CSS::GridRepeat grid_repeat)
-    : m_type(Type::Repeat)
-    , m_grid_repeat(grid_repeat)
-{
-}
-
-ExplicitGridTrack::ExplicitGridTrack(CSS::GridSize grid_size)
-    : m_type(Type::Default)
-    , m_grid_size(grid_size)
+ExplicitGridTrack::ExplicitGridTrack(Variant<GridFitContent, GridRepeat, GridMinMax, GridSize>&& value)
+    : m_value(move(value))
 {
 }
 
 String ExplicitGridTrack::to_string() const
 {
-    switch (m_type) {
-    case Type::FitContent:
-        return m_grid_fit_content.to_string();
-    case Type::MinMax:
-        return m_grid_minmax.to_string();
-    case Type::Repeat:
-        return m_grid_repeat.to_string();
-    case Type::Default:
-        return m_grid_size.to_string();
-    default:
-        VERIFY_NOT_REACHED();
-    }
+    return m_value.visit([](auto const& track) {
+        return track.to_string();
+    });
 }
 
 String GridLineNames::to_string() const
diff --git a/Libraries/LibWeb/CSS/GridTrackSize.h b/Libraries/LibWeb/CSS/GridTrackSize.h
index 9fbc30dd2e6..f344f20d0f7 100644
--- a/Libraries/LibWeb/CSS/GridTrackSize.h
+++ b/Libraries/LibWeb/CSS/GridTrackSize.h
@@ -167,67 +167,25 @@ private:
 
 class ExplicitGridTrack {
 public:
-    enum class Type {
-        FitContent,
-        MinMax,
-        Repeat,
-        Default,
-    };
-    ExplicitGridTrack(CSS::GridFitContent);
-    ExplicitGridTrack(CSS::GridRepeat);
-    ExplicitGridTrack(CSS::GridMinMax);
-    ExplicitGridTrack(CSS::GridSize);
+    ExplicitGridTrack(Variant<GridFitContent, GridRepeat, GridMinMax, GridSize>&& value);
 
-    bool is_fit_content() const { return m_type == Type::FitContent; }
-    GridFitContent fit_content() const
-    {
-        VERIFY(is_fit_content());
-        return m_grid_fit_content;
-    }
+    bool is_fit_content() const { return m_value.has<GridFitContent>(); }
+    GridFitContent const& fit_content() const { return m_value.get<GridFitContent>(); }
 
-    bool is_repeat() const { return m_type == Type::Repeat; }
-    GridRepeat repeat() const
-    {
-        VERIFY(is_repeat());
-        return m_grid_repeat;
-    }
+    bool is_repeat() const { return m_value.has<GridRepeat>(); }
+    GridRepeat const& repeat() const { return m_value.get<GridRepeat>(); }
 
-    bool is_minmax() const { return m_type == Type::MinMax; }
-    GridMinMax minmax() const
-    {
-        VERIFY(is_minmax());
-        return m_grid_minmax;
-    }
+    bool is_minmax() const { return m_value.has<GridMinMax>(); }
+    GridMinMax const& minmax() const { return m_value.get<GridMinMax>(); }
 
-    bool is_default() const { return m_type == Type::Default; }
-    GridSize grid_size() const
-    {
-        VERIFY(is_default());
-        return m_grid_size;
-    }
-
-    Type type() const { return m_type; }
+    bool is_default() const { return m_value.has<GridSize>(); }
+    GridSize const& grid_size() const { return m_value.get<GridSize>(); }
 
     String to_string() const;
-    bool operator==(ExplicitGridTrack const& other) const
-    {
-        if (is_fit_content() && other.is_fit_content())
-            return m_grid_fit_content == other.fit_content();
-        if (is_repeat() && other.is_repeat())
-            return m_grid_repeat == other.repeat();
-        if (is_minmax() && other.is_minmax())
-            return m_grid_minmax == other.minmax();
-        if (is_default() && other.is_default())
-            return m_grid_size == other.grid_size();
-        return false;
-    }
+    bool operator==(ExplicitGridTrack const& other) const = default;
 
 private:
-    Type m_type;
-    GridFitContent m_grid_fit_content;
-    GridRepeat m_grid_repeat;
-    GridMinMax m_grid_minmax;
-    GridSize m_grid_size;
+    Variant<GridFitContent, GridRepeat, GridMinMax, GridSize> m_value;
 };
 
 }
diff --git a/Libraries/LibWeb/Layout/GridFormattingContext.cpp b/Libraries/LibWeb/Layout/GridFormattingContext.cpp
index f468f0b9c42..b759a5687bf 100644
--- a/Libraries/LibWeb/Layout/GridFormattingContext.cpp
+++ b/Libraries/LibWeb/Layout/GridFormattingContext.cpp
@@ -577,18 +577,13 @@ void GridFormattingContext::initialize_grid_tracks_from_definition(GridDimension
                 repeat_count = track_definition.repeat().repeat_count();
         }
         for (auto _ = 0; _ < repeat_count; _++) {
-            switch (track_definition.type()) {
-            case CSS::ExplicitGridTrack::Type::Default:
-            case CSS::ExplicitGridTrack::Type::FitContent:
-            case CSS::ExplicitGridTrack::Type::MinMax:
+            if (track_definition.is_default() || track_definition.is_fit_content() || track_definition.is_minmax()) {
                 tracks.append(GridTrack::create_from_definition(track_definition));
-                break;
-            case CSS::ExplicitGridTrack::Type::Repeat:
+            } else if (track_definition.is_repeat()) {
                 for (auto& explicit_grid_track : track_definition.repeat().grid_track_size_list().track_list()) {
                     tracks.append(GridTrack::create_from_definition(explicit_grid_track));
                 }
-                break;
-            default:
+            } else {
                 VERIFY_NOT_REACHED();
             }
         }