Bug 64008

Summary: Remove unused parameter from _add_test in test_expectations.py
Product: WebKit Reporter: Adam Barth <abarth>
Component: New BugsAssignee: Adam Barth <abarth>
Status: RESOLVED FIXED    
Severity: Normal CC: dpranke, eric, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

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.