Bug 182043

Summary: Update paths of CSS tests from WPT
Product: WebKit Reporter: Manuel Rego Casasnovas <rego>
Component: Tools / TestsAssignee: Manuel Rego Casasnovas <rego>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, lforschler, webkit-bug-importer, youennf
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch for landing none

Manuel Rego Casasnovas
Reported 2018-01-24 02:22:26 PST
The CSS test suite folders in WPT has been renamed to remove the version number from them. It'd be nice to update them in the WebKit import, to make it easier to import and synchronize tests in the future.
Attachments
Patch (6.90 MB, patch)
2018-01-24 02:24 PST, Manuel Rego Casasnovas 		no flags
DetailsFormatted DiffDiff
Patch for landing (6.67 MB, patch)
2018-01-24 07:17 PST, Manuel Rego Casasnovas 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Manuel Rego Casasnovas
Comment 1 2018-01-24 02:24:52 PST
Created attachment 332141 [details] Patch
youenn fablet
Comment 2 2018-01-24 06:26:15 PST
Comment on attachment 332141 [details] Patch I am not sure listing all the files in the ChangeLog helps much. Can we remove the exact list, maybe just listing the important folders?
Manuel Rego Casasnovas
Comment 3 2018-01-24 07:17:14 PST
Created attachment 332157 [details] Patch for landing
Manuel Rego Casasnovas
Comment 4 2018-01-24 07:21:26 PST
(In reply to youenn fablet from comment #2) > Comment on attachment 332141 [details] > Patch > > I am not sure listing all the files in the ChangeLog helps much. Yeah that was done automatically. > Can we remove the exact list, maybe just listing the important folders? I've done that. Thanks for the review.
Manuel Rego Casasnovas
Comment 5 2018-01-24 07:54:07 PST
Committed r227521: <https://trac.webkit.org/changeset/227521>
Radar WebKit Bug Importer
Comment 6 2018-01-24 07:56:26 PST
<rdar://problem/36824672>
Note You need to log in before you can comment on or make changes to this bug.