Fieldset: Handle column properties correctly.

mstensho · chromium-wpt-export-bot · commit d4246a84f280 · 2025-03-19T00:56:55.000-07:00
Propagate column properties correctly to the anonymous fieldset content box. If column-count or column-width is auto, ComputedStyle's ColumnCount() or ColumnWidth() are 0, respectively. So check if they are non-auto before setting the values on the anonymous content box. At the same time, there's no need to set the properties to their initial values if no multicol is going to take place. Change-Id: I8ec6d69222a9d2359a5d34a08627258362cf8592 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6368877 Commit-Queue: Morten Stenshorne <mstensho@chromium.org> Reviewed-by: Ian Kilpatrick <ikilpatrick@chromium.org> Cr-Commit-Position: refs/heads/main@{#1434633}
diff --git a/html/rendering/non-replaced-elements/the-fieldset-and-legend-elements/fieldset-multicol-only-column-count.html b/html/rendering/non-replaced-elements/the-fieldset-and-legend-elements/fieldset-multicol-only-column-count.html
@@ -0,0 +1,30 @@
+<!DOCTYPE html>
+<title>fieldset multicol with auto count, non-auto width</title>
+<link rel="help" href="https://html.spec.whatwg.org/multipage/rendering.html#the-fieldset-and-legend-elements">
+<script src="/resources/testharness.js"></script>
+<script src="/resources/testharnessreport.js"></script>
+<style>
+ #test { margin: 0; padding: 0; border: none }
+ #test, #ref { width:min-content; column-count:4; }
+ p { margin: 0 }
+</style>
+<fieldset id="test">
+  <p>1</p>
+  <p>2</p>
+  <p>3</p>
+  <p>4</p>
+  <p>5</p>
+</fieldset>
+<div id="ref">
+  <p>1</p>
+  <p>2</p>
+  <p>3</p>
+  <p>4</p>
+  <p>5</p>
+</div>
+<script>
+  test(() => {
+    assert_equals(getComputedStyle(document.getElementById('test')).height,
+                  getComputedStyle(document.getElementById('ref')).height);
+  });
+</script>
diff --git a/html/rendering/non-replaced-elements/the-fieldset-and-legend-elements/fieldset-multicol-only-column-width.html b/html/rendering/non-replaced-elements/the-fieldset-and-legend-elements/fieldset-multicol-only-column-width.html
@@ -0,0 +1,30 @@
+<!DOCTYPE html>
+<title>fieldset multicol with auto count, non-auto width</title>
+<link rel="help" href="https://html.spec.whatwg.org/multipage/rendering.html#the-fieldset-and-legend-elements">
+<script src="/resources/testharness.js"></script>
+<script src="/resources/testharnessreport.js"></script>
+<style>
+ #test { margin: 0; padding: 0; border: none }
+ #test, #ref { width:500px; column-width:100px; }
+ p { margin: 0 }
+</style>
+<fieldset id="test">
+  <p>1</p>
+  <p>2</p>
+  <p>3</p>
+  <p>4</p>
+  <p>5</p>
+</fieldset>
+<div id="ref">
+  <p>1</p>
+  <p>2</p>
+  <p>3</p>
+  <p>4</p>
+  <p>5</p>
+</div>
+<script>
+  test(() => {
+    assert_equals(getComputedStyle(document.getElementById('test')).height,
+                  getComputedStyle(document.getElementById('ref')).height);
+  });
+</script>
