RESOLVED FIXED 205926
[css-grid] Import grid-align-content-distribution tests to WPT folder
https://bugs.webkit.org/show_bug.cgi?id=205926
Summary [css-grid] Import grid-align-content-distribution tests to WPT folder
rmonteriso
Reported 2020-01-08 07:28:27 PST
Following tests have been moved, as per https://github.com/web-platform-tests/wpt/pull/20110: * grid-align-content-distribution-vertical-lr.html * grid-align-content-distribution-vertical-rl.html * grid-align-content-distribution.html Their corresponding outdated tests in fast/css-grid-layout have been removed.
Attachments
Patch (168.61 KB, patch)
2020-01-08 07:43 PST, rmonteriso
no flags
Patch (228.73 KB, patch)
2020-01-09 01:52 PST, rmonteriso
no flags
Patch (70.79 KB, patch)
2020-01-15 06:26 PST, rmonteriso
no flags
Patch (71.65 KB, patch)
2020-01-15 07:03 PST, rmonteriso
no flags
Patch (73.36 KB, patch)
2020-01-17 04:32 PST, rmonteriso
no flags
Patch (2.59 KB, patch)
2020-01-17 05:13 PST, rmonteriso
no flags
Patch (169.28 KB, patch)
2020-01-17 05:49 PST, rmonteriso
no flags
Patch (175.95 KB, patch)
2020-01-19 05:51 PST, rmonteriso
no flags
rmonteriso
Comment 1 2020-01-08 07:43:40 PST
rmonteriso
Comment 2 2020-01-09 01:52:01 PST
Javier Fernandez
Comment 3 2020-01-09 13:22:54 PST
Comment on attachment 387201 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=387201&action=review > LayoutTests/imported/w3c/web-platform-tests/css/css-grid/alignment/grid-align-content-distribution-expected.txt:52 > +FAIL .grid 1 assert_equals: These tests has some cases that fail. We should investigate it before landing. > LayoutTests/imported/w3c/web-platform-tests/css/css-grid/alignment/grid-align-content-distribution-vertical-lr-expected.txt:52 > +FAIL .grid 1 assert_equals: Ditto > LayoutTests/imported/w3c/web-platform-tests/css/css-grid/alignment/grid-align-content-distribution-vertical-lr.html:8 > +<link rel="stylesheet" href="/css/support/grid.css"> I think this test needs the alignment.css support file. Perhaps this is the cause of the failures mentioned before. > LayoutTests/imported/w3c/web-platform-tests/css/css-grid/alignment/grid-align-content-distribution-vertical-rl-expected.txt:52 > +FAIL .grid 1 assert_equals: Ditto > LayoutTests/imported/w3c/web-platform-tests/css/css-grid/alignment/grid-align-content-distribution-vertical-rl.html:8 > +<link rel="stylesheet" href="/css/support/grid.css"> Ditto > LayoutTests/imported/w3c/web-platform-tests/css/css-grid/alignment/grid-align-content-distribution.html:8 > +<link rel="stylesheet" href="/css/support/grid.css"> Ditto
rmonteriso
Comment 4 2020-01-15 06:26:19 PST
rmonteriso
Comment 5 2020-01-15 07:03:47 PST
rmonteriso
Comment 6 2020-01-17 04:32:38 PST
rmonteriso
Comment 7 2020-01-17 05:13:22 PST
rmonteriso
Comment 8 2020-01-17 05:49:58 PST
rmonteriso
Comment 9 2020-01-19 05:51:42 PST
WebKit Commit Bot
Comment 10 2020-01-20 01:48:54 PST
The commit-queue encountered the following flaky tests while processing attachment 388183 [details]: editing/spelling/spellcheck-attribute.html bug 206178 (authors: g.czajkowski@samsung.com, mark.lam@apple.com, and rniwa@webkit.org) The commit-queue is continuing to process your patch.
WebKit Commit Bot
Comment 11 2020-01-20 01:49:40 PST
Comment on attachment 388183 [details] Patch Clearing flags on attachment: 388183 Committed r254816: <https://trac.webkit.org/changeset/254816>
WebKit Commit Bot
Comment 12 2020-01-20 01:49:41 PST
All reviewed patches have been landed. Closing bug.
Radar WebKit Bug Importer
Comment 13 2020-01-20 01:50:20 PST
Note You need to log in before you can comment on or make changes to this bug.