RESOLVED FIXED 235102
[css-flexbox] Import WPT tests - January edition
https://bugs.webkit.org/show_bug.cgi?id=235102
Summary [css-flexbox] Import WPT tests - January edition
Sergio Villar Senin
Reported 2022-01-12 02:28:24 PST
[css-flexbox] Import WPT tests - January edition
Attachments
Patch (208.96 KB, patch)
2022-01-12 02:33 PST, Sergio Villar Senin
no flags
Patch (180.89 KB, patch)
2022-01-12 04:02 PST, Sergio Villar Senin
rego: review+
Sergio Villar Senin
Comment 1 2022-01-12 02:33:13 PST
Manuel Rego Casasnovas
Comment 2 2022-01-12 03:43:44 PST
Comment on attachment 448918 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=448918&action=review > LayoutTests/imported/w3c/resources/resource-files.json:22 > + "css/css-flexbox/abspos/flex-abspos-staticpos-align-self-safe-001-ref.html", Mmmm, there are a lot of -ref files here, but they're actually not being imported. Maybe you forgot to git add them. The problem is that since r287267 we're importing the -ref.html files, and also creating copies as -expected.html. So we end up having duplicates. I'm not sure what's the best solution, but one option is to revert that patch to run the import script, and generate a patch that doesn't add so many -ref. Other option might be to merge duplicated files (like it has already happened in some folders). See this comment https://bugs.webkit.org/show_bug.cgi?id=203784#c27 for more info.
Sergio Villar Senin
Comment 3 2022-01-12 04:02:25 PST
Manuel Rego Casasnovas
Comment 4 2022-01-12 07:41:59 PST
Comment on attachment 448930 [details] Patch r=me
Sergio Villar Senin
Comment 5 2022-01-12 08:56:29 PST
Radar WebKit Bug Importer
Comment 6 2022-01-12 08:57:15 PST
Note You need to log in before you can comment on or make changes to this bug.