Bug 226064 - [LayoutTests] Delete unused LayoutTests/fast resources
Summary: [LayoutTests] Delete unused LayoutTests/fast resources
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: Jonathan Bedard
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2021-05-20 19:31 PDT by Jonathan Bedard
Modified: 2021-05-25 15:29 PDT (History)
3 users (show)

See Also:


Attachments
Patch (806.89 KB, patch)
2021-05-20 19:52 PDT, Jonathan Bedard
no flags Details | Formatted Diff | Diff
Patch (489.17 KB, patch)
2021-05-21 08:52 PDT, Jonathan Bedard
no flags Details | Formatted Diff | Diff
Patch (106.68 KB, patch)
2021-05-21 09:51 PDT, Jonathan Bedard
no flags Details | Formatted Diff | Diff
Patch (106.10 KB, patch)
2021-05-21 10:15 PDT, Jonathan Bedard
no flags Details | Formatted Diff | Diff
Patch (105.47 KB, patch)
2021-05-21 12:23 PDT, Jonathan Bedard
dewei_zhu: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jonathan Bedard 2021-05-20 19:31:09 PDT
There are a number of resources in LayoutTests/fast that are unused. We should delete them.
Comment 1 Radar WebKit Bug Importer 2021-05-20 19:31:27 PDT
<rdar://problem/78289833>
Comment 2 Jonathan Bedard 2021-05-20 19:52:51 PDT
Created attachment 429261 [details]
Patch
Comment 3 Alexey Proskuryakov 2021-05-20 20:52:18 PDT
This is surprising. All tests here were manually written, but there are so many unused resources?
Comment 4 Jonathan Bedard 2021-05-21 08:52:22 PDT
Created attachment 429294 [details]
Patch
Comment 5 Jonathan Bedard 2021-05-21 09:51:13 PDT
Created attachment 429299 [details]
Patch
Comment 6 Jonathan Bedard 2021-05-21 10:15:45 PDT
Created attachment 429305 [details]
Patch
Comment 7 Jonathan Bedard 2021-05-21 12:23:16 PDT
Created attachment 429322 [details]
Patch
Comment 8 Jonathan Bedard 2021-05-21 13:07:06 PDT
There are a few image files that can be deleted as well, but I can't put those in a patch.
Comment 9 Jonathan Bedard 2021-05-25 12:33:46 PDT
Landed 238130@main (r278033)