Bug 58384

Summary:

Update kill-old-processes to include a few new processes

Product:

WebKit

Reporter:

Lucas Forschler <lforschler>

Component:

Tools / Tests

Assignee:

Lucas Forschler <lforschler>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

aroben, commit-queue

Priority:

Version:

528+ (Nightly build)

Hardware:

OS:

OS X 10.5

Attachments:

Description	Flags
update the kill-old-processes script	none

Description Lucas Forschler 2011-04-12 15:43:07 PDT

Currently the kill-old-processes does not kill
Safari.exe
devenv.com

others?

update the tool to catch these.

Comment 1 Adam Roben (:aroben) 2011-04-12 15:47:06 PDT

WebKitTestRunner.exe
WebKit2WebProcess.exe

Comment 2 Adam Roben (:aroben) 2011-04-12 15:48:01 PDT

I guess those are already on there!

Comment 3 Lucas Forschler 2011-04-12 16:04:27 PDT

Created attachment 89295 [details]
update the kill-old-processes script

Add VCBuildHelper.exe, Safari.exe, and devenv.com to the list.

Comment 4 Adam Roben (:aroben) 2011-04-13 05:49:49 PDT

Comment on attachment 89295 [details]
update the kill-old-processes script

A followup patch to sort the list of tasks would be great.

Comment 5 Lucas Forschler 2011-04-13 05:52:21 PDT

Also, I think we should add the /T parameter to taskkill.  This will kill any CHILD processes that were created by the parent.

Comment 6 WebKit Commit Bot 2011-04-13 07:28:43 PDT

Comment on attachment 89295 [details]
update the kill-old-processes script

Clearing flags on attachment: 89295

Committed r83732: <http://trac.webkit.org/changeset/83732>

Comment 7 WebKit Commit Bot 2011-04-13 07:28:52 PDT

All reviewed patches have been landed.  Closing bug.