[IFC][Ruby] Fix fast/ruby/ruby-text-before-after-content.html

https://bugs.webkit.org/show_bug.cgi?id=264221

Reviewed by Antti Koivisto.

Sum of the individual inline items' logical width may not match the final content width due
to certain inline layout rules like white-space collapsing.
Let's just get the content width off of the line when finished placing the ruby base content.

* Source/WebCore/layout/formattingContexts/inline/ruby/RubyFormattingContext.cpp:
(WebCore::Layout::RubyFormattingContext::layoutRubyBaseInlineAxis): (turn for loop into a while and handle content alignment outside of the loop).

Canonical link: https://commits.webkit.org/270245@main

Author: alanbaradlay

Source/WebCore/layout/formattingContexts/inline/ruby/RubyFormattingContext.cpp

@@ -132,25 +132,22 @@ RubyFormattingContext::BaseLayoutResult RubyFormattingContext::layoutRubyBaseInl
 {
     // Append ruby base content (including start/end inline box) to the line and apply "ruby-align: space-around" on the ruby subrange.
     auto& formattingUtils = parentFormattingContext().formattingUtils();
-    auto lineLogicalRight = line.contentLogicalRight();
-    auto baseContentLogicalWidth = InlineLayoutUnit { };
     auto baseRunStart = line.runs().size();
+    auto baseContentLogicalLeft = line.contentLogicalRight();
 
-    for (size_t index = rubyBaseContentStartIndex; index < inlineItemList.size(); ++index) {
-        auto& rubyBaseInlineItem = inlineItemList[index];
-        if (&rubyBaseInlineItem.layoutBox() == &rubyBaseLayoutBox) {
-            auto baseRunCount = line.runs().size() - baseRunStart;
-            auto logicalRightSpacing = InlineLayoutUnit { };
-            if (baseRunCount)
-                logicalRightSpacing = applyRubyAlign(line, { baseRunStart, baseRunStart + baseRunCount }, rubyBaseLayoutBox, baseContentLogicalWidth);
-            return { index - rubyBaseContentStartIndex, logicalRightSpacing };
-        }
-        auto logicalWidth = formattingUtils.inlineItemWidth(rubyBaseInlineItem, lineLogicalRight + baseContentLogicalWidth, { });
-        line.append(rubyBaseInlineItem, rubyBaseInlineItem.style(), logicalWidth);
-        baseContentLogicalWidth += logicalWidth;
+    auto index = rubyBaseContentStartIndex;
+    while (index < inlineItemList.size() && &inlineItemList[index].layoutBox() != &rubyBaseLayoutBox) {
+        auto& rubyBaseContentItem = inlineItemList[index++];
+        line.append(rubyBaseContentItem, rubyBaseContentItem.style(), formattingUtils.inlineItemWidth(rubyBaseContentItem, line.contentLogicalRight(), { }));
+    }
+    ASSERT(index < inlineItemList.size());
+    auto logicalRightSpacing = InlineLayoutUnit { };
+    auto baseRunCount = line.runs().size() - baseRunStart;
+    if (baseRunCount) {
+        auto baseContentLogicalWidth = line.contentLogicalRight() - baseContentLogicalLeft;
+        logicalRightSpacing = applyRubyAlign(line, { baseRunStart, baseRunStart + baseRunCount }, rubyBaseLayoutBox, baseContentLogicalWidth);
     }
-    ASSERT_NOT_REACHED();
-    return { inlineItemList.size() - rubyBaseContentStartIndex, { } };
+    return { index - rubyBaseContentStartIndex, logicalRightSpacing };
 }
 
 InlineLayoutPoint RubyFormattingContext::placeAnnotationBox(const Box& rubyBaseLayoutBox)