From 299a9dd643dbea60cd9c9d7e68c63e6d1409347b Mon Sep 17 00:00:00 2001 From: Morten Stenshorne Date: Mon, 6 Mar 2023 14:04:33 +0000 Subject: [PATCH] Bug 1820123 [wpt PR 38804] - Disallow breaks before trailing border/padding in table cells., a=testonly Automatic update from web-platform-tests Disallow breaks before trailing border/padding in table cells. We normally allow a last-resort break between the content box block-end and any block-end border/padding, if that can help avoid fragmentainer overflow. Don't do this for table cells, as that upsets the row stretching machinery. See how PlaceCells() is rerun in NGTableRowLayoutAlgorithm::Layout() with a new row size. This is a simple fix. A more elaborate fix would be to store a flag in the break token, saying that we're past the content box (we already have one for being past the *border* box), and just keep on allowing such breaks before trailing border/padding. But it just doesn't seem worth it at this point. Bug: 1420637 Change-Id: Ie43f3af898bab5cbf9a062232383adf307376ced Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4303193 Commit-Queue: Morten Stenshorne Reviewed-by: Ian Kilpatrick Cr-Commit-Position: refs/heads/main@{#1113046} -- wpt-commits: d031b37d079e789288f76164cd44b8c556d79d68 wpt-pr: 38804 --- .../css-break/table/cell-large-bottom-padding-crash.html | 6 ++++++ 1 file changed, 6 insertions(+) create mode 100644 testing/web-platform/tests/css/css-break/table/cell-large-bottom-padding-crash.html diff --git a/testing/web-platform/tests/css/css-break/table/cell-large-bottom-padding-crash.html b/testing/web-platform/tests/css/css-break/table/cell-large-bottom-padding-crash.html new file mode 100644 index 000000000000..e87df1fe7a22 --- /dev/null +++ b/testing/web-platform/tests/css/css-break/table/cell-large-bottom-padding-crash.html @@ -0,0 +1,6 @@ + + + +
+
boo
+