From a8e7d294c626dceb6091583ff95015428ba9f94b Mon Sep 17 00:00:00 2001 From: Morten Stenshorne Date: Wed, 27 Oct 2021 07:06:15 -0700 Subject: [PATCH] Only inflate contenteditable blocks at the first fragment. If a block is empty, it will usually fit inside one fragmentainer. But not always. If there's tall borders/padding, for instance, we might split the block into fragments. Never attempt to stretch the intrinsic block-size of a fragment unless we're working on the first one, or else we might end up creating an infinite number of them. Bug: 1261747 Change-Id: Ic9bc5c08259d39b39e3c7a5b6a49b197d1dd115a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3245260 Reviewed-by: Ian Kilpatrick Commit-Queue: Morten Stenshorne Cr-Commit-Position: refs/heads/main@{#935412} --- css/css-break/fragmented-empty-contenteditable-crash.html | 7 +++++++ 1 file changed, 7 insertions(+) create mode 100644 css/css-break/fragmented-empty-contenteditable-crash.html diff --git a/css/css-break/fragmented-empty-contenteditable-crash.html b/css/css-break/fragmented-empty-contenteditable-crash.html new file mode 100644 index 00000000000000..c2149c77d709bd --- /dev/null +++ b/css/css-break/fragmented-empty-contenteditable-crash.html @@ -0,0 +1,7 @@ + + + +

Test passes if it doesn't crash (or freeze).

+
+
+