Bug 226796

Summary: [LayoutTests] Delete unused LayoutTests/js resources
Product: WebKit Reporter: Robert Jenner <jenner>
Component: Tools / TestsAssignee: Robert Jenner <jenner>
Status: RESOLVED FIXED    
Severity: Normal CC: jbedard, webkit-bot-watchers-bugzilla, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch
none
Patch none

Robert Jenner
Reported 2021-06-08 17:08:55 PDT
There are a number of php resources in LayoutTests/js that are unused by any test.
Attachments
Patch (5.42 KB, patch)
2021-06-08 17:24 PDT, Robert Jenner 		no flags
DetailsFormatted DiffDiff
Patch (5.28 KB, patch)
2021-06-10 15:13 PDT, Robert Jenner 		no flags
DetailsFormatted DiffDiff
Patch (2.91 KB, patch)
2021-06-10 15:27 PDT, Robert Jenner 		no flags
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Robert Jenner
Comment 1 2021-06-08 17:24:38 PDT
Created attachment 430921 [details] Patch
Radar WebKit Bug Importer
Comment 2 2021-06-08 17:38:37 PDT
<rdar://problem/79045844>
Robert Jenner
Comment 3 2021-06-10 15:08:31 PDT
(In reply to Robert Jenner from comment #0) > There are a number of php resources in LayoutTests/js that are unused by any > test. Correction to this. This applies to all un-used resources, not just un-used php resources.
Robert Jenner
Comment 4 2021-06-10 15:13:53 PDT
Created attachment 431140 [details] Patch
Jonathan Bedard
Comment 5 2021-06-10 15:23:26 PDT
Comment on attachment 431140 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=431140&action=review > LayoutTests/js/kde/resources/README:-1 > -======================================================================== Should keep the README.
Robert Jenner
Comment 6 2021-06-10 15:27:52 PDT
Created attachment 431142 [details] Patch
EWS
Comment 7 2021-06-11 16:27:01 PDT
Committed r278791 (238748@main): <https://commits.webkit.org/238748@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 431142 [details].
Note You need to log in before you can comment on or make changes to this bug.