Bug 235685

Summary: Add fuzzy match data to WPT
Product: WebKit Reporter: Jon Lee <jonlee>
Component: Tools / TestsAssignee: Jon Lee <jonlee>
Status: RESOLVED FIXED    
Severity: Normal CC: clopez, darin, ews-watchlist, webkit-bug-importer, youennf
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://github.com/web-platform-tests/wpt/pull/32692
Bug Depends on:    
Bug Blocks: 231828    
Attachments:
Description Flags
Patch
none
Patch
none
Patch
none
Patch
none
Patch
none
Patch
none
Patch
none
Patch
none
Patch
none
Patch
none
Patch
none
Patch
none
Patch
none
Patch none

Jon Lee
Reported 2022-01-26 20:42:21 PST
.
Attachments
Patch (66.05 KB, patch)
2022-01-27 00:36 PST, Jon Lee
no flags
Patch (66.05 KB, patch)
2022-01-27 00:58 PST, Jon Lee
no flags
Patch (66.02 KB, patch)
2022-01-27 09:46 PST, Jon Lee
no flags
Patch (69.15 KB, patch)
2022-01-28 17:45 PST, Jon Lee
no flags
Patch (64.25 KB, patch)
2022-02-03 23:24 PST, Jon Lee
no flags
Patch (64.25 KB, patch)
2022-02-03 23:30 PST, Jon Lee
no flags
Patch (64.25 KB, patch)
2022-02-03 23:32 PST, Jon Lee
no flags
Patch (64.25 KB, patch)
2022-02-03 23:33 PST, Jon Lee
no flags
Patch (64.25 KB, patch)
2022-02-03 23:36 PST, Jon Lee
no flags
Patch (64.25 KB, patch)
2022-02-03 23:38 PST, Jon Lee
no flags
Patch (64.25 KB, patch)
2022-02-03 23:41 PST, Jon Lee
no flags
Patch (64.25 KB, patch)
2022-02-03 23:41 PST, Jon Lee
no flags
Patch (64.25 KB, patch)
2022-02-03 23:44 PST, Jon Lee
no flags
Patch (64.25 KB, patch)
2022-02-03 23:45 PST, Jon Lee
no flags
Jon Lee
Comment 1 2022-01-27 00:36:29 PST
EWS Watchlist
Comment 2 2022-01-27 00:38:14 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
Jon Lee
Comment 3 2022-01-27 00:58:35 PST
Jon Lee
Comment 4 2022-01-27 09:46:14 PST
Darin Adler
Comment 5 2022-01-28 10:56:57 PST
Comment on attachment 450148 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=450148&action=review > LayoutTests/imported/w3c/ChangeLog:3 > + Add fuzzy match data to WPT How will upstreaming of this work? Is this cross-platform setting that we all have to agree on? Will this be obliterated when we update to newer versions of WPT?
Darin Adler
Comment 6 2022-01-28 10:57:22 PST
Comment on attachment 450148 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=450148&action=review >> LayoutTests/imported/w3c/ChangeLog:3 >> + Add fuzzy match data to WPT > > How will upstreaming of this work? Is this cross-platform setting that we all have to agree on? Will this be obliterated when we update to newer versions of WPT? I meant "a cross-browser setting"
Jon Lee
Comment 7 2022-01-28 12:02:33 PST
(In reply to Darin Adler from comment #5) > Comment on attachment 450148 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=450148&action=review > > > LayoutTests/imported/w3c/ChangeLog:3 > > + Add fuzzy match data to WPT > > How will upstreaming of this work? Is this cross-platform setting that we > all have to agree on? Will this be obliterated when we update to newer > versions of WPT? I'm going to adjust the close tags for some of these tests, but as part of it will begin the process of upstreaming the changes. It is a cross-browser setting that everyone needs to agree on, hence the 0 lower bound.
Jon Lee
Comment 8 2022-01-28 12:03:03 PST
(In reply to Jon Lee from comment #7) > I'm going to adjust the close tags for some of these tests, but as part of > it will begin the process of upstreaming the changes. Meaning I'm going to upload a new patch.
Jon Lee
Comment 9 2022-01-28 17:45:06 PST
Radar WebKit Bug Importer
Comment 10 2022-02-02 20:43:18 PST
Darin Adler
Comment 11 2022-02-03 04:29:00 PST
Comment on attachment 450296 [details] Patch Need a rebased patch
Jon Lee
Comment 12 2022-02-03 22:58:22 PST
Jon Lee
Comment 13 2022-02-03 22:59:08 PST
Submitted a patch that pulls the latest version of WPT test to put this patch on top of.
Jon Lee
Comment 14 2022-02-03 23:24:56 PST
Jon Lee
Comment 15 2022-02-03 23:30:47 PST
Jon Lee
Comment 16 2022-02-03 23:32:25 PST
Jon Lee
Comment 17 2022-02-03 23:33:07 PST
Jon Lee
Comment 18 2022-02-03 23:36:00 PST
Jon Lee
Comment 19 2022-02-03 23:38:12 PST
Jon Lee
Comment 20 2022-02-03 23:41:05 PST
Jon Lee
Comment 21 2022-02-03 23:41:50 PST
Jon Lee
Comment 22 2022-02-03 23:44:43 PST
Jon Lee
Comment 23 2022-02-03 23:45:35 PST
Jon Lee
Comment 24 2022-02-03 23:45:56 PST
Submitted web-platform-tests pull request: https://github.com/web-platform-tests/wpt/pull/32692
Jon Lee
Comment 25 2022-02-04 09:18:09 PST
The test failure on Mac-wk1 appears to not be related to the patch. It flakily fails on the EWS bot. https://results.webkit.org/?suite=layout-tests&test=inspector/debugger/breakpoints/resolved-dump-all-pause-locations.html
Jon Lee
Comment 26 2022-02-04 16:07:28 PST
Changeling found its way in the wrong place, oops. But it's in there! Committed r289141: <https://commits.webkit.org/r289141>
Note You need to log in before you can comment on or make changes to this bug.