Bug 226796 - [LayoutTests] Delete unused LayoutTests/js resources
Summary: [LayoutTests] Delete unused LayoutTests/js 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: Robert Jenner
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2021-06-08 17:08 PDT by Robert Jenner
Modified: 2021-06-11 16:27 PDT (History)
3 users (show)

See Also:

Attachments
Patch (5.42 KB, patch)
2021-06-08 17:24 PDT, Robert Jenner 		no flags Details | Formatted Diff | Diff
Patch (5.28 KB, patch)
2021-06-10 15:13 PDT, Robert Jenner 		no flags Details | Formatted Diff | Diff
Patch (2.91 KB, patch)
2021-06-10 15:27 PDT, Robert Jenner 		no flags Details | Formatted Diff | Diff
Show Obsolete (2) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Robert Jenner 2021-06-08 17:08:55 PDT 
There are a number of php resources in LayoutTests/js that are unused by any test.
Comment 1 Robert Jenner 2021-06-08 17:24:38 PDT 
Created attachment 430921 [details]
Patch
Comment 2 Radar WebKit Bug Importer 2021-06-08 17:38:37 PDT 
<rdar://problem/79045844>
Comment 3 Robert Jenner 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.
Comment 4 Robert Jenner 2021-06-10 15:13:53 PDT 
Created attachment 431140 [details]
Patch
Comment 5 Jonathan Bedard 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.
Comment 6 Robert Jenner 2021-06-10 15:27:52 PDT 
Created attachment 431142 [details]
Patch
Comment 7 EWS 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].