Summary: | [NRWT] Remove TestArgs class | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Hayato Ito <hayato> | ||||||
Component: | Tools / Tests | Assignee: | Nobody <webkit-unassigned> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | dpranke, eric, mihaip, ojan, tony | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | PC | ||||||||
OS: | OS X 10.5 | ||||||||
Bug Depends on: | 53004 | ||||||||
Bug Blocks: | 51091, 53071 | ||||||||
Attachments: |
|
Description
Hayato Ito
2011-01-24 21:55:28 PST
Created attachment 80019 [details]
remove-testargs
Yup, looks good to me (but I'm not a reviewer). adding more people who are reviewers ... Comment on attachment 80019 [details]
remove-testargs
I fully support deleting code.
This patch (A) depends on other patch (B), https://bugs.webkit.org/show_bug.cgi?id=53004, which is under the review. There is no strong relation between (A) and (B), but we should wait for patch (B) to apply this patch (A). I'll set review flag and update this bugzilla entry once patch (B) is landed. Created attachment 81188 [details]
remove-testargs--review-ready
The patch is now ready to be reviewed. Comment on attachment 81188 [details]
remove-testargs--review-ready
OK.
Committed r77606: <http://trac.webkit.org/changeset/77606> |