Bug 236101 - Fix WPT test importer and resync web-platform-tests/workers from upstream
Summary: Fix WPT test importer and resync web-platform-tests/workers from upstream
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: Chris Dumez
URL:
Keywords: InRadar
Depends on:
Blocks: 235958
  Show dependency treegraph
 
Reported: 2022-02-03 13:36 PST by Chris Dumez
Modified: 2022-02-04 08:37 PST (History)
11 users (show)

See Also:


Attachments
Patch (158.24 KB, patch)
2022-02-03 13:41 PST, Chris Dumez
no flags Details | Formatted Diff | Diff
Patch (158.68 KB, patch)
2022-02-03 13:47 PST, Chris Dumez
no flags Details | Formatted Diff | Diff
Patch (173.70 KB, patch)
2022-02-03 14:07 PST, Chris Dumez
no flags Details | Formatted Diff | Diff
Patch (173.71 KB, patch)
2022-02-03 14:16 PST, Chris Dumez
no flags Details | Formatted Diff | Diff
Patch (164.68 KB, patch)
2022-02-03 14:32 PST, Chris Dumez
no flags Details | Formatted Diff | Diff
Patch (166.69 KB, patch)
2022-02-03 14:39 PST, Chris Dumez
no flags Details | Formatted Diff | Diff
Patch (166.64 KB, patch)
2022-02-04 07:35 PST, Chris Dumez
no flags Details | Formatted Diff | Diff
Patch (169.72 KB, patch)
2022-02-04 07:49 PST, Chris Dumez
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Chris Dumez 2022-02-03 13:36:14 PST
Fix WPT test importer and resync web-platform-tests/workers from upstream.
Comment 1 Chris Dumez 2022-02-03 13:41:18 PST
Created attachment 450811 [details]
Patch
Comment 2 Chris Dumez 2022-02-03 13:47:22 PST
Created attachment 450814 [details]
Patch
Comment 3 Chris Dumez 2022-02-03 14:07:55 PST
Created attachment 450818 [details]
Patch
Comment 4 Chris Dumez 2022-02-03 14:16:17 PST
Created attachment 450820 [details]
Patch
Comment 5 Chris Dumez 2022-02-03 14:32:17 PST
Created attachment 450823 [details]
Patch
Comment 6 Chris Dumez 2022-02-03 14:39:52 PST
Created attachment 450824 [details]
Patch
Comment 7 youenn fablet 2022-02-04 07:31:35 PST
Comment on attachment 450824 [details]
Patch

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

> Tools/Scripts/webkitpy/w3c/test_importer.py:194
> +            window=('html',),

s/html',/html'/ here and below.
Comment 8 Chris Dumez 2022-02-04 07:35:46 PST
Created attachment 450895 [details]
Patch
Comment 9 Chris Dumez 2022-02-04 07:49:43 PST
Created attachment 450897 [details]
Patch
Comment 10 Chris Dumez 2022-02-04 07:51:17 PST
(In reply to youenn fablet from comment #7)
> Comment on attachment 450824 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=450824&action=review
> 
> > Tools/Scripts/webkitpy/w3c/test_importer.py:194
> > +            window=('html',),
> 
> s/html',/html'/ here and below.

Turns out this is required in Python to make sure it gets recognized as a tuple.
I tried but it caused webkitpy tests to fail so I went back to my original code.
Comment 11 Chris Dumez 2022-02-04 08:36:01 PST
Comment on attachment 450897 [details]
Patch

Clearing flags on attachment: 450897

Committed r289116 (246813@trunk): <https://commits.webkit.org/246813@trunk>
Comment 12 Chris Dumez 2022-02-04 08:36:04 PST
All reviewed patches have been landed.  Closing bug.
Comment 13 Radar WebKit Bug Importer 2022-02-04 08:37:20 PST
<rdar://problem/88490743>