[webkit-reviews] review requested: [Bug 9794] Teach
run-webkit-tests how to ignore tests with performance
improvements : [Attachment 9280] Patch v1
bugzilla-request-daemon at opendarwin.org
bugzilla-request-daemon at opendarwin.org
Sat Jul 8 20:27:30 PDT 2006
David Kilzer (ddkilzer) <ddkilzer at kilzer.net> has asked for review:
Bug 9794: Teach run-webkit-tests how to ignore tests with performance
improvements
http://bugzilla.opendarwin.org/show_bug.cgi?id=9794
Attachment 9280: Patch v1
http://bugzilla.opendarwin.org/attachment.cgi?id=9280&action=edit
------- Additional Comments from David Kilzer (ddkilzer) <ddkilzer at kilzer.net>
Changes in Patch v1:
- Added --ignore-tests command-line switch. Takes a comma-separated list of
tests and test directories to ignore (exclude) when running run-webkit-tests.
Added processIgnoreTests() subroutine to process the arguments.
- Added --help command-line switch with usage statement.
- Moved call to setConfiguration() before GetOptions() so that
--debug/--release arguments are removed from @ARGV before calling
GetOptions(). Keep result from call for use with build-dumprendertree later.
- Check result of GetOptions() and exit run-webkit-tests if invalid arguments
are used.
- Made --singly imply --verbose so all of the "$verbose || $singly" checks
could be reduced to "$verbose".
- Switched from using the find(1) command via subshell to using the File::Find
Perl module. Directory filtering and file filtering are now expressed as
anonymous subroutines rather than obscure find(1) command-line arguments. It's
faster, too!
- Miscellaneous minor clean up.
More information about the webkit-reviews
mailing list