From 12363845db18d06ffb2197afd7e2ec37c0d5222d Mon Sep 17 00:00:00 2001 From: Vladimir Levin Date: Tue, 2 Aug 2022 09:48:11 -0700 Subject: [PATCH] CIS: Apply contain-intrinsic-size: auto to content-visibility: hidden as well Based on discussion on https://github.com/w3c/csswg-drafts/issues/6308#issuecomment-1189673809 and based on the spec as written, contain-intrinsic-size auto should apply to all elements that skip their contents, not just content-visibility: auto elements. This patch does this. R=chrishtr@chromium.org Change-Id: Ibf38be014ae25ff64af027f6c1608caa7ded3514 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3777917 Reviewed-by: Oriol Brufau Commit-Queue: Vladimir Levin Cr-Commit-Position: refs/heads/main@{#1030610} --- css/css-sizing/contain-intrinsic-size/auto-005.html | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/css/css-sizing/contain-intrinsic-size/auto-005.html b/css/css-sizing/contain-intrinsic-size/auto-005.html index 4a920ac80600652..2b7e03192dc701e 100644 --- a/css/css-sizing/contain-intrinsic-size/auto-005.html +++ b/css/css-sizing/contain-intrinsic-size/auto-005.html @@ -1,6 +1,6 @@ -Tests that contain-intrinsic-size: auto only works with content-visibility: auto, not just hidden visibility +Tests that contain-intrinsic-size: auto also works with content-visibility: hidden @@ -17,8 +17,7 @@ el.style.contain = "size"; el.style.contentVisibility = "hidden"; log.innerText = el.offsetHeight; - // No content-visibility: auto, so this should remain 1. - assert_equals(el.offsetHeight, 1); + assert_equals(el.offsetHeight, 50); } var t = async_test("contain-intrinsic-size: auto");