Bug 212920 - [css-grid] Import some grid tests from WPT
Summary: [css-grid] Import some grid tests from WPT
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Javier Fernandez
URL:
Keywords: InRadar
Depends on:
Blocks: 209282
  Show dependency treegraph
 
Reported: 2020-06-08 14:09 PDT by Javier Fernandez
Modified: 2020-06-09 05:12 PDT (History)
7 users (show)

See Also:


Attachments
Patch (46.08 KB, patch)
2020-06-08 15:23 PDT, Javier Fernandez
no flags Details | Formatted Diff | Diff
Patch (67.88 KB, patch)
2020-06-09 01:59 PDT, Javier Fernandez
no flags Details | Formatted Diff | Diff
Patch (49.34 KB, patch)
2020-06-09 03:00 PDT, Javier Fernandez
no flags Details | Formatted Diff | Diff
Patch (49.34 KB, patch)
2020-06-09 03:38 PDT, Javier Fernandez
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Javier Fernandez 2020-06-08 14:09:51 PDT
We need to import a few tests from the WPT repository to cover some cases related to how baseline alignment may affect the grid's intrinsic size computation.

These tests are needed to cover some of the changes to fix bug #209282
Comment 1 Javier Fernandez 2020-06-08 14:11:42 PDT
These tests has been merged in the PR https://github.com/web-platform-tests/wpt/pull/24022
Comment 2 Javier Fernandez 2020-06-08 15:23:05 PDT
Created attachment 401382 [details]
Patch
Comment 3 EWS Watchlist 2020-06-08 15:23:48 PDT
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 4 Manuel Rego Casasnovas 2020-06-09 01:47:51 PDT
Comment on attachment 401382 [details]
Patch

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

> LayoutTests/TestExpectations:1070
> +webkit.org/b/209282 imported/w3c/web-platform-tests/css/css-grid/layout-algorithm/baseline-alignment-affects-intrinsic-size-001.html [ Failure ]

I don't understand why you mark them as Failure. They have a -expected.txt file, so it won't need to be marked like that. And also the -expected file has all PASS.
Comment 5 Javier Fernandez 2020-06-09 01:59:32 PDT
Created attachment 401427 [details]
Patch

Change expectations to reflect the test cases failing
Comment 6 Manuel Rego Casasnovas 2020-06-09 02:07:47 PDT
Comment on attachment 401427 [details]
Patch

r=me, but the corresponding w3c-import.log file should be updated too.
Comment 7 Manuel Rego Casasnovas 2020-06-09 02:08:07 PDT
Also wait for green EWSs before landing.
Comment 8 Javier Fernandez 2020-06-09 03:00:05 PDT
Created attachment 401431 [details]
Patch

Tests pass on mac platform, so reflect that in the test expectations
Comment 9 Javier Fernandez 2020-06-09 03:38:47 PDT
Created attachment 401432 [details]
Patch

Fixed text expectations
Comment 10 Javier Fernandez 2020-06-09 04:57:25 PDT
Comment on attachment 401432 [details]
Patch

The failures in the mac-debug-wk1 seems unrelated to this change, so landing.
Comment 11 EWS 2020-06-09 05:11:18 PDT
Committed r262788: <https://trac.webkit.org/changeset/262788>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 401432 [details].
Comment 12 Radar WebKit Bug Importer 2020-06-09 05:12:18 PDT
<rdar://problem/64160279>