LibWeb: Let FFC parent context "handle" sizing of child FFC container

When we have nested flexbox layouts within one another, and the child
context wants to call up to the parent context and ask for help with
dimensioning the child flex container, we now simply do nothing.

As far as I can tell, this works out just fine, since the child flex
container will already be dimensioned by the flex layout algorithm.

2 files changed, 28 insertions, 5 deletions

diff --git a/Userland/Libraries/LibWeb/Layout/FlexFormattingContext.cpp b/Userland/Libraries/LibWeb/Layout/FlexFormattingContext.cpp
index cd34662c42..45b8b8af3d 100644
--- a/Userland/Libraries/LibWeb/Layout/FlexFormattingContext.cpp
+++ b/Userland/Libraries/LibWeb/Layout/FlexFormattingContext.cpp
@@ -768,12 +768,36 @@ float FlexFormattingContext::content_based_minimum_size(FlexItem const& item) co
     return unclamped_size;
 }
 
+bool FlexFormattingContext::can_determine_size_of_child() const
+{
+    return true;
+}
+
+void FlexFormattingContext::determine_width_of_child(Box const&, AvailableSpace const&)
+{
+    // NOTE: For now, we simply do nothing here. If a child context is calling up to us
+    //       and asking us to determine its width, we've already done so as part of the
+    //       flex layout algorithm.
+}
+
+void FlexFormattingContext::determine_height_of_child(Box const&, AvailableSpace const&)
+{
+    // NOTE: For now, we simply do nothing here. If a child context is calling up to us
+    //       and asking us to determine its height, we've already done so as part of the
+    //       flex layout algorithm.
+}
+
 // https://drafts.csswg.org/css-flexbox-1/#algo-main-container
 void FlexFormattingContext::determine_main_size_of_flex_container()
 {
     // Determine the main size of the flex container using the rules of the formatting context in which it participates.
     // NOTE: The automatic block size of a block-level flex container is its max-content size.
 
+    // FIXME: The code below doesn't know how to size absolutely positioned flex containers at all.
+    //        We just leave it alone for now and let the parent context deal with it.
+    if (flex_container().is_absolutely_positioned())
+        return;
+
     // FIXME: Once all parent contexts now how to size a given child, we can remove
     //        `can_determine_size_of_child()`.
     if (parent()->can_determine_size_of_child()) {
@@ -785,11 +809,6 @@ void FlexFormattingContext::determine_main_size_of_flex_container()
         return;
     }
 
-    // HACK: The hack below doesn't know how to size absolutely positioned flex containers at all.
-    //       We just leave it alone for now and let the parent context deal with it.
-    if (flex_container().is_absolutely_positioned())
-        return;
-
     if (is_row_layout()) {
         if (!flex_container().is_out_of_flow(*parent()) && m_state.get(*flex_container().containing_block()).has_definite_width()) {
             set_main_size(flex_container(), calculate_stretch_fit_width(flex_container(), m_available_space_for_flex_container->space.width));
diff --git a/Userland/Libraries/LibWeb/Layout/FlexFormattingContext.h b/Userland/Libraries/LibWeb/Layout/FlexFormattingContext.h
index bedafc54fe..70e68c34e4 100644
--- a/Userland/Libraries/LibWeb/Layout/FlexFormattingContext.h
+++ b/Userland/Libraries/LibWeb/Layout/FlexFormattingContext.h
@@ -23,6 +23,10 @@ public:
 
     Box const& flex_container() const { return context_box(); }
 
+    virtual bool can_determine_size_of_child() const override;
+    virtual void determine_width_of_child(Box const&, AvailableSpace const&) override;
+    virtual void determine_height_of_child(Box const&, AvailableSpace const&) override;
+
 private:
     void dump_items() const;