WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
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
Details
Formatted Diff
Diff
Patch
(180.89 KB, patch)
2022-01-12 04:02 PST
,
Sergio Villar Senin
rego
: review+
Details
Formatted Diff
Diff
Show Obsolete
(1)
View All
Add attachment
proposed patch, testcase, etc.
Sergio Villar Senin
Comment 1
2022-01-12 02:33:13 PST
Created
attachment 448918
[details]
Patch
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
Created
attachment 448930
[details]
Patch
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
Committed
r287927
(?): <
https://commits.webkit.org/r287927
>
Radar WebKit Bug Importer
Comment 6
2022-01-12 08:57:15 PST
<
rdar://problem/87461795
>
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug