Bug 111037

Summary: PerfTestRunner doesn't need _needs_http and _has_http_lock
Product: WebKit Reporter: Ryosuke Niwa <rniwa>
Component: Tools / TestsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED WONTFIX    
Severity: Normal CC: abarth, benjamin, dpranke
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 77037    
Attachments:
Description Flags
Cleanup none

Ryosuke Niwa
Reported 2013-02-27 21:22:09 PST
PerfTestRunner has _has_http_lock and _has_http_lock but they're only used in run() and two functions that are called by run(). We can easily eliminate these member variables.
Attachments
Cleanup (3.11 KB, patch)
2013-02-27 21:25 PST, Ryosuke Niwa 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Ryosuke Niwa
Comment 1 2013-02-27 21:25:39 PST
Created attachment 190647 [details] Cleanup
Adam Barth
Comment 2 2013-02-27 23:43:02 PST
Comment on attachment 190647 [details] Cleanup ok
Note You need to log in before you can comment on or make changes to this bug.