Skip to content

Commit 5f81c60

Browse files
marco-cmarco-c
committed
Bug 1873102 [wpt PR 43858] - Do not round the clipping region with mixed overflow clip, a=testonly
Automatic update from web-platform-tests Do not round the clipping region with mixed overflow clip The CSSWG resolved [1] to not apply rounded border clipping when one of overflow-x or overflow-y computes to clip and the other computes to visible (https://drafts.csswg.org/css-overflow/#corner-clipping). This patch updates our implementation to not create the inner border radius clip in this mixed clip/visible scenario. [1] w3c/csswg-drafts#7434 Bug: 1416649 Change-Id: I4d24ac82ecdafba5fa691efb1ce85892c734c717 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5171405 Reviewed-by: David Baron Commit-Queue: Philip Rogers Cr-Commit-Position: refs/heads/main{#1243240} -- wpt-commits: 7e7527ccba8afd472bd49972a4ba0ae0c58208ac wpt-pr: 43858 UltraBlame original commit: e60370c109d6e6f1b765260a2230504ae6371e89
1 parent b29ad32 commit 5f81c60

File tree

4 files changed

+804
-0
lines changed

4 files changed

+804
-0
lines changed

‎testing/web-platform/tests/css/css-overflow/rounded-overflow-clip-visible-ref.html

Lines changed: 99 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,99 @@
1+
<
2+
!
3+
DOCTYPE
4+
html
5+
>
6+
<
7+
style
8+
>
9+
.
10+
box
11+
{
12+
width
13+
:
14+
100px
15+
;
16+
height
17+
:
18+
100px
19+
;
20+
background
21+
:
22+
green
23+
;
24+
display
25+
:
26+
inline
27+
-
28+
block
29+
;
30+
}
31+
<
32+
/
33+
style
34+
>
35+
There
36+
should
37+
be
38+
4
39+
100x100
40+
green
41+
squares
42+
(
43+
no
44+
rounded
45+
corners
46+
)
47+
below
48+
.
49+
<
50+
br
51+
>
52+
<
53+
div
54+
class
55+
=
56+
"
57+
box
58+
"
59+
>
60+
<
61+
/
62+
div
63+
>
64+
<
65+
div
66+
class
67+
=
68+
"
69+
box
70+
"
71+
>
72+
<
73+
/
74+
div
75+
>
76+
<
77+
div
78+
class
79+
=
80+
"
81+
box
82+
"
83+
>
84+
<
85+
/
86+
div
87+
>
88+
<
89+
div
90+
class
91+
=
92+
"
93+
box
94+
"
95+
>
96+
<
97+
/
98+
div
99+
>

‎testing/web-platform/tests/css/css-overflow/rounded-overflow-clip-visible.html

Lines changed: 294 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,294 @@
1+
<
2+
!
3+
DOCTYPE
4+
html
5+
>
6+
<
7+
title
8+
>
9+
Border
10+
radius
11+
should
12+
not
13+
round
14+
the
15+
clipping
16+
region
17+
when
18+
mixing
19+
overflow
20+
:
21+
visible
22+
and
23+
clip
24+
<
25+
/
26+
title
27+
>
28+
<
29+
link
30+
rel
31+
=
32+
"
33+
help
34+
"
35+
href
36+
=
37+
"
38+
https
39+
:
40+
/
41+
/
42+
drafts
43+
.
44+
csswg
45+
.
46+
org
47+
/
48+
css
49+
-
50+
overflow
51+
/
52+
#
53+
corner
54+
-
55+
clipping
56+
"
57+
>
58+
<
59+
link
60+
rel
61+
=
62+
"
63+
help
64+
"
65+
href
66+
=
67+
"
68+
https
69+
:
70+
/
71+
/
72+
github
73+
.
74+
com
75+
/
76+
w3c
77+
/
78+
csswg
79+
-
80+
drafts
81+
/
82+
issues
83+
/
84+
7434
85+
"
86+
>
87+
<
88+
link
89+
rel
90+
=
91+
"
92+
match
93+
"
94+
href
95+
=
96+
"
97+
rounded
98+
-
99+
overflow
100+
-
101+
clip
102+
-
103+
visible
104+
-
105+
ref
106+
.
107+
html
108+
"
109+
>
110+
<
111+
style
112+
>
113+
.
114+
container
115+
{
116+
width
117+
:
118+
100px
119+
;
120+
height
121+
:
122+
50px
123+
;
124+
overflow
125+
:
126+
clip
127+
visible
128+
;
129+
background
130+
:
131+
red
132+
;
133+
display
134+
:
135+
inline
136+
-
137+
block
138+
;
139+
}
140+
.
141+
border
142+
-
143+
radius
144+
{
145+
border
146+
-
147+
radius
148+
:
149+
25px
150+
;
151+
}
152+
.
153+
child
154+
{
155+
width
156+
:
157+
200px
158+
;
159+
height
160+
:
161+
100px
162+
;
163+
background
164+
:
165+
green
166+
;
167+
fill
168+
:
169+
green
170+
;
171+
}
172+
<
173+
/
174+
style
175+
>
176+
There
177+
should
178+
be
179+
4
180+
100x100
181+
green
182+
squares
183+
(
184+
no
185+
rounded
186+
corners
187+
)
188+
below
189+
.
190+
<
191+
br
192+
>
193+
<
194+
div
195+
class
196+
=
197+
"
198+
container
199+
border
200+
-
201+
radius
202+
"
203+
>
204+
<
205+
div
206+
class
207+
=
208+
"
209+
child
210+
"
211+
>
212+
<
213+
/
214+
div
215+
>
216+
<
217+
/
218+
div
219+
>
220+
<
221+
div
222+
class
223+
=
224+
"
225+
container
226+
"
227+
>
228+
<
229+
div
230+
class
231+
=
232+
"
233+
child
234+
"
235+
>
236+
<
237+
/
238+
div
239+
>
240+
<
241+
/
242+
div
243+
>
244+
<
245+
svg
246+
class
247+
=
248+
"
249+
container
250+
border
251+
-
252+
radius
253+
"
254+
>
255+
<
256+
rect
257+
class
258+
=
259+
"
260+
child
261+
"
262+
>
263+
<
264+
/
265+
rect
266+
>
267+
<
268+
/
269+
svg
270+
>
271+
<
272+
svg
273+
class
274+
=
275+
"
276+
container
277+
"
278+
>
279+
<
280+
rect
281+
class
282+
=
283+
"
284+
child
285+
"
286+
>
287+
<
288+
/
289+
rect
290+
>
291+
<
292+
/
293+
svg
294+
>

0 commit comments

Comments
 (0)