Bug 218565

Summary: [css-grid] Move tests grid-columns-rows-get-set-multiple and grid-content-sized-columns-resolution to WPT folder
Product: WebKit Reporter: zsun
Component: CSSAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: clopez, ews-watchlist, jfernandez, rego, svillar, webkit-bug-importer, youennf
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch
none
Patch
none
Patch none

Description zsun 2020-11-04 07:14:28 PST
This patch moves the following tests

* grid-columns-rows-get-set-multiple.html
* grid-content-sized-columns-resolution.html

to WPT folder and removes their corresponding outdated versions from css-grid-layout folder
Comment 1 zsun 2020-11-05 01:57:45 PST
Created attachment 413273 [details]
Patch
Comment 2 EWS Watchlist 2020-11-05 01:58:58 PST
This patch modifies the imported WPT tests. Please ensure that any changes on the tests (not coming from a WPT import) are exported to WPT. Please see https://trac.webkit.org/wiki/WPTExportProcess
Comment 3 zsun 2020-11-05 02:01:40 PST
Created attachment 413274 [details]
Patch
Comment 4 Manuel Rego Casasnovas 2020-11-05 02:10:22 PST
Comment on attachment 413274 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=413274&action=review

Thanks for doing this, just a minor change is required, the rest looks good to me.

> LayoutTests/ChangeLog:7
> +

I'd add here a small comment, explaining that these tests have been moved upstream in a Chromium change, linking the change or the WPT PR generated by the change.

> LayoutTests/imported/w3c/ChangeLog:12
> +

I believe we miss a change in "css/css-grid/parsing/w3c-import.log" to include the new tests.
Comment 6 zsun 2020-11-05 02:35:12 PST
Created attachment 413278 [details]
Patch
Comment 7 zsun 2020-11-05 03:01:57 PST
Created attachment 413280 [details]
Patch
Comment 8 Manuel Rego Casasnovas 2020-11-05 03:06:57 PST
Comment on attachment 413280 [details]
Patch

r=me, thanks!

Let's wait for the EWSs to be green before landing.
Comment 9 EWS 2020-11-05 05:11:25 PST
Committed r269426: <https://trac.webkit.org/changeset/269426>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 413280 [details].
Comment 10 Radar WebKit Bug Importer 2020-11-05 05:12:29 PST
<rdar://problem/71074866>