Skip to content

Commit

Permalink
Bug 1843439 [wpt PR 41025] - CV/CIS: content-visibility: auto forces …
Browse files Browse the repository at this point in the history
…contain-intrinsic-size: auto, a=testonly

Automatic update from web-platform-tests
CV/CIS: content-visibility: auto forces contain-intrinsic-size: auto

This patch implements a resolution on
w3c/csswg-drafts#8407 (comment)

With a follow-up resolution that this should happen at computed value
time (same issue).

I2P&S is here:
https://groups.google.com/a/chromium.org/g/blink-dev/c/aYpmS8hP2UQ/m/CO4wGRiYCQAJ

Note that the current implementation sets this behind an experimental
flag, pending I2S approval.

R=andruudchromium.org

Bug: 1418453
Change-Id: I06ad14460c1e620b3f35937d09a3317c563905e0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4685406
Commit-Queue: Vladimir Levin <vmpstrchromium.org>
Reviewed-by: Khushal Sagar <khushalsagarchromium.org>
Cr-Commit-Position: refs/heads/main{#1170651}

--

wpt-commits: c6fb50b0e11397aceaff70ddae8bd08c036ac4c5
wpt-pr: 41025

UltraBlame original commit: 56052b674c85aed2057def6ebf9c6b864895897f
  • Loading branch information
marco-c committed Jul 24, 2023
1 parent d59bbd4 commit cb3339f
Show file tree
Hide file tree
Showing 5 changed files with 131 additions and 56 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -127,15 +127,15 @@
{
height
:
1000px
10000px
;
}
.
large_child
{
height
:
20000px
40000px
;
position
:
Expand All @@ -160,6 +160,9 @@
>
<
div
id
=
e1
class
=
auto
Expand All @@ -168,7 +171,7 @@
div
class
=
small_child
large_child
>
<
/
Expand All @@ -180,6 +183,9 @@
>
<
div
id
=
e2
class
=
auto
Expand All @@ -188,7 +194,7 @@
div
class
=
small_child
large_child
>
<
/
Expand All @@ -200,6 +206,9 @@
>
<
div
id
=
e3
class
=
auto
Expand Down Expand Up @@ -231,6 +240,9 @@
>
<
div
id
=
e4
class
=
auto
Expand All @@ -251,6 +263,9 @@
>
<
div
id
=
e5
class
=
auto
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -176,15 +176,15 @@
size
:
1px
1000px
10000px
;
}
.
child
{
height
:
20000px
40000px
;
position
:
Expand All @@ -203,52 +203,55 @@
0
;
}
.
before_target
{
height
:
40000px
;
}
<
/
style
>
<
div
class
id
=
auto
>
<
div
e1
class
=
child
>
<
/
div
"
auto
before_target
"
>
<
/
div
>
<
div
class
id
=
auto
>
<
div
e2
class
=
child
>
<
/
div
"
auto
before_target
"
>
<
/
div
>
<
div
id
=
e3
class
=
auto
Expand Down Expand Up @@ -280,6 +283,9 @@
>
<
div
id
=
e4
class
=
auto
Expand All @@ -300,6 +306,9 @@
>
<
div
id
=
e5
class
=
auto
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -178,15 +178,15 @@
size
:
1px
1000px
10000px
;
}
.
child
{
height
:
20000px
40000px
;
position
:
Expand All @@ -205,6 +205,14 @@
0
;
}
.
before_target
{
height
:
40000px
;
}
<
/
style
Expand All @@ -213,17 +221,10 @@
div
class
=
"
auto
>
<
div
class
=
child
>
<
/
div
before_target
"
>
<
/
Expand All @@ -233,17 +234,10 @@
div
class
=
"
auto
>
<
div
class
=
child
>
<
/
div
before_target
"
>
<
/
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -1563,12 +1563,51 @@
(
)
;
/
/
Note
that
because
content
-
visibility
:
auto
implies
contain
-
intrinsic
-
size
:
auto
/
/
the
rest
of
the
expected
heights
are
kVisibleHeight
since
that
will
be
the
last
/
/
remembered
size
.
assert_equals
(
vis
.
clientHeight
kHiddenHeight
kVisibleHeight
"
with
toggle
Expand Down Expand Up @@ -1630,7 +1669,7 @@
vis
.
clientHeight
kHiddenHeight
kVisibleHeight
"
with
toggle
Expand Down Expand Up @@ -1659,7 +1698,7 @@
vis
.
clientHeight
kHiddenHeight
kVisibleHeight
"
with
toggle
Expand Down Expand Up @@ -1718,7 +1757,7 @@
vis
.
clientHeight
kHiddenHeight
kVisibleHeight
"
with
toggle
Expand Down
Loading

0 comments on commit cb3339f

Please sign in to comment.