Skip to content
This repository was archived by the owner on Jul 9, 2025. It is now read-only.

Commit f3a5c52

Browse files
utpathakmoz-wptsync-bot
authored andcommitted
Bug 1932834 [wpt PR 49329] - Fix failing WPTs introduced by https://crrev.com/c/5937916, a=testonly
Automatic update from web-platform-tests Fix failing WPTs introduced by https://crrev.com/c/5937916 There are few failures introduced by WPT import in the editing component where the editing includes forward-delete command alongwith the white-space CSS properties. This CL fixes the failing WPTs by setting the expectation as per the CSS specification linked below. https: //www.w3.org/TR/css-text-3/#white-space-property Bug: 373924023 Change-Id: I80d322fbafa9d3a592d80d78c43480f2e34b8485 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5953176 Reviewed-by: Koji Ishii <[email protected]> Reviewed-by: Siye Liu <[email protected]> Commit-Queue: Utkarsh Pathak <[email protected]> Cr-Commit-Position: refs/heads/main@{#1386719} -- wpt-commits: f69c9d92fbdbcacda06f85d3c609281f8f07462f wpt-pr: 49329
1 parent 3003673 commit f3a5c52

File tree

1 file changed

+26
-26
lines changed

1 file changed

+26
-26
lines changed

testing/web-platform/tests/editing/other/forwarddelete-before-invisible-line-break.html

Lines changed: 26 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -32,94 +32,94 @@
3232
for (const data of [
3333
{
3434
initialInnerHTML: "<p>a[] <br></p>",
35-
expected: "<p>a</p>",
35+
expected: collapseWhiteSpaces ? "<p>a <br></p>" : "<p>a<br></p>",
3636
},
3737
{
3838
initialInnerHTML: "<p>a[ ]<br></p>",
39-
expected: "<p>a</p>",
39+
expected: collapseWhiteSpaces ? "<p>a <br></p>" : "<p>a<br></p>",
4040
},
4141
{
4242
initialInnerHTML: "<p>a []b<br></p>",
4343
expected: collapseWhiteSpaces
44-
? (useBR ? ["<p>a <br></p>", "<p>a&nbsp;</p>"] : ["<p>a <br></p>", "<p>a \n</p>", "<p>a&nbsp;</p>"])
45-
: "<p>a </p>",
44+
? (useBR ? ["<p>a <br></p>", "<p>a&nbsp;<br></p>"] : ["<p>a <br></p>", "<p>a \n</p>", "<p>a&nbsp;<br></p>"])
45+
: "<p>a <br></p>",
4646
},
4747
{
4848
initialInnerHTML: "<p>a [b]<br></p>",
4949
expected: collapseWhiteSpaces
50-
? (useBR ? ["<p>a <br></p>", "<p>a&nbsp;</p>"] : ["<p>a <br></p>", "<p>a \n</p>", "<p>a&nbsp;</p>"])
51-
: "<p>a </p>",
50+
? (useBR ? ["<p>a <br></p>", "<p>a&nbsp;<br></p>"] : ["<p>a <br></p>", "<p>a \n</p>", "<p>a&nbsp;<br></p>"])
51+
: "<p>a <br></p>",
5252
},
5353
{
5454
initialInnerHTML: "<p>a[] \n</p>",
55-
expected: "<p>a</p>",
55+
expected: collapseWhiteSpaces ? "<p>a \n</p>" : "<p>a\n</p>",
5656
skipIf: () => useBR,
5757
},
5858
{
5959
initialInnerHTML: "<p>a[ ]\n</p>",
60-
expected: "<p>a</p>",
60+
expected: collapseWhiteSpaces ? "<p>a \n</p>" : "<p>a\n</p>",
6161
skipIf: () => useBR,
6262
},
6363
{
6464
initialInnerHTML: "<p>a []b\n</p>",
6565
expected: collapseWhiteSpaces
66-
? ["<p>a \n</p>", "<p>a&nbsp;</p>"]
67-
: "<p>a </p>",
66+
? ["<p>a \n</p>", "<p>a&nbsp;\n</p>"]
67+
: "<p>a \n</p>",
6868
skipIf: () => useBR,
6969
},
7070
{
7171
initialInnerHTML: "<p>a [b]\n</p>",
7272
expected: collapseWhiteSpaces
73-
? ["<p>a \n</p>", "<p>a&nbsp;</p>"]
74-
: "<p>a </p>",
73+
? ["<p>a \n</p>", "<p>a&nbsp;\n</p>"]
74+
: "<p>a \n</p>",
7575
skipIf: () => useBR,
7676
},
7777
{
7878
initialInnerHTML: "<div>a[] <br><div>b</div></div>",
79-
expected: "<div>a<div>b</div></div>",
79+
expected: collapseWhiteSpaces ? "ab" : "<div>a<br><div>b</div></div>",
8080
},
8181
{
8282
initialInnerHTML: "<div>a[ ]<br><div>b</div></div>",
83-
expected: "<div>a<div>b</div></div>",
83+
expected: collapseWhiteSpaces ? "ab" : "<div>a<br><div>b</div></div>",
8484
},
8585
{
8686
initialInnerHTML: "<div>a []b<br><div>b</div></div>",
8787
expected: collapseWhiteSpaces
8888
? (useBR
89-
? ["<div>a <br><div>b</div></div>", "<div>a&nbsp;<div>b</div></div>"]
90-
: ["<div>a <br><div>b</div></div>", "<div>a \n<div>b</div></div>", "<div>a&nbsp;<div>b</div></div>"])
91-
: "<div>a <div>b</div></div>",
89+
? ["<div>a <br><div>b</div></div>", "<div>a&nbsp;<br><div>b</div></div>"]
90+
: ["<div>a <br><div>b</div></div>", "<div>a \n<div>b</div></div>", "<div>a&nbsp;<br><div>b</div></div>"])
91+
: "<div>a <br><div>b</div></div>",
9292
},
9393
{
9494
initialInnerHTML: "<div>a [b]<br><div>b</div></div>",
9595
expected: collapseWhiteSpaces
9696
? (useBR
97-
? ["<div>a <br><div>b</div></div>", "<div>a&nbsp;<div>b</div></div>"]
98-
: ["<div>a <br><div>b</div></div>", "<div>a \n<div>b</div></div>", "<div>a&nbsp;<div>b</div></div>"])
99-
: "<div>a <div>b</div></div>",
97+
? ["<div>a <br><div>b</div></div>", "<div>a&nbsp;<br><div>b</div></div>"]
98+
: ["<div>a <br><div>b</div></div>", "<div>a \n<div>b</div></div>", "<div>a&nbsp;<br><div>b</div></div>"])
99+
: "<div>a <br><div>b</div></div>",
100100
},
101101
{
102102
initialInnerHTML: "<div>a[] \n<div>b</div></div>",
103-
expected: "<div>a<div>b</div></div>",
103+
expected: collapseWhiteSpaces ? "ab" : "<div>a\n<div>b</div></div>",
104104
skipIf: () => useBR,
105105
},
106106
{
107107
initialInnerHTML: "<div>a[ ]\n<div>b</div></div>",
108-
expected: "<div>a<div>b</div></div>",
108+
expected: collapseWhiteSpaces ? "ab" : "<div>a\n<div>b</div></div>",
109109
skipIf: () => useBR,
110110
},
111111
{
112112
initialInnerHTML: "<div>a []b\n<div>b</div></div>",
113113
expected: collapseWhiteSpaces
114-
? ["<div>a \n<div>b</div></div>", "<div>a&nbsp;<div>b</div></div>"]
115-
: "<div>a <div>b</div></div>",
114+
? ["<div>a \n<div>b</div></div>", "<div>a&nbsp;\n<div>b</div></div>"]
115+
: "<div>a \n<div>b</div></div>",
116116
skipIf: () => useBR,
117117
},
118118
{
119119
initialInnerHTML: "<div>a [b]\n<div>b</div></div>",
120120
expected: collapseWhiteSpaces
121-
? ["<div>a \n<div>b</div></div>", "<div>a&nbsp;<div>b</div></div>"]
122-
: "<div>a <div>b</div></div>",
121+
? ["<div>a \n<div>b</div></div>", "<div>a&nbsp;\n<div>b</div></div>"]
122+
: "<div>a \n<div>b</div></div>",
123123
skipIf: () => useBR,
124124
},
125125
]) {

0 commit comments

Comments
 (0)