RESOLVED FIXED 64008
Remove unused parameter from _add_test in test_expectations.py 
https://bugs.webkit.org/show_bug.cgi?id=64008
Summary Remove unused parameter from _add_test in test_expectations.py
Adam Barth
Reported 2011-07-06 11:27:11 PDT
Remove unused parameter from _add_test in test_expectations.py
Attachments
Patch (4.14 KB, patch)
2011-07-06 11:27 PDT, Adam Barth 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Adam Barth
Comment 1 2011-07-06 11:27:34 PDT
Created attachment 99854 [details] Patch
Eric Seidel (no email)
Comment 2 2011-07-06 11:36:44 PDT
Comment on attachment 99854 [details] Patch OK.
WebKit Review Bot
Comment 3 2011-07-06 12:07:52 PDT
Comment on attachment 99854 [details] Patch Clearing flags on attachment: 99854 Committed r90479: <http://trac.webkit.org/changeset/90479>
WebKit Review Bot
Comment 4 2011-07-06 12:07:56 PDT
All reviewed patches have been landed. Closing bug.
Dirk Pranke
Comment 5 2011-07-06 13:48:30 PDT
It would probably be good to remove the rest of the "num_matches" logic at this point, since that code (which was already disabled) is now even more disabled :).
Adam Barth
Comment 6 2011-07-06 13:54:22 PDT
(In reply to comment #5) > It would probably be good to remove the rest of the "num_matches" logic at this point, since that code (which was already disabled) is now even more disabled :). I didn't follow that rabbit, but it's good to know that none if it is used.
Note You need to log in before you can comment on or make changes to this bug.