Bug 165173
| Summary: | run-benchmark should have a switch to avoid killing browser process unconditionally | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Carlos Alberto Lopez Perez <clopez> |
| Component: | Tools / Tests | Assignee: | Nobody <webkit-unassigned> |
| Status: | NEW | ||
| Severity: | Normal | CC: | dewei_zhu, lforschler, rniwa |
| Priority: | P2 | ||
| Version: | WebKit Nightly Build | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| Bug Depends on: | |||
| Bug Blocks: | 164792 | ||
Carlos Alberto Lopez Perez
After r209090 <http://trac.webkit.org/changeset/209090> run-benchmark in Linux doesn't execute a "killall $browser" when it is asked to run the benchmarks with browser $browser. It instead keeps track of the browser PID it executed and only kill this PID.
This is a desired behaviour when the user is testing on the machine. However for a performance bot running tests automatically it isn't.
After discussing this in bug 164792 it seems we can add a command line switch to avoid issuing the killall, so users that want to avoid this behaviour of the script can just activate the switch. And at the same time we can keep as default behaviour killing all process with name $browser.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Ryosuke Niwa
(In reply to comment #0)
> After r209090 <http://trac.webkit.org/changeset/209090> run-benchmark in
> Linux doesn't execute a "killall $browser" when it is asked to run the
> benchmarks with browser $browser. It instead keeps track of the browser PID
> it executed and only kill this PID.
>
> This is a desired behaviour when the user is testing on the machine. However
> for a performance bot running tests automatically it isn't.
>
> After discussing this in bug 164792 it seems we can add a command line
> switch to avoid issuing the killall, so users that want to avoid this
> behaviour of the script can just activate the switch. And at the same time
> we can keep as default behaviour killing all process with name $browser.
Alternatively, we can add an option like --kill-all to mean kill all existing browser instances to be used on bots.