Bug 58384 - Update kill-old-processes to include a few new processes
Summary: Update kill-old-processes to include a few new processes
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC OS X 10.5
: P2 Normal
Assignee: Lucas Forschler
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-04-12 15:43 PDT by Lucas Forschler
Modified: 2011-04-13 07:28 PDT (History)
2 users (show)

See Also:


Attachments
update the kill-old-processes script (1.70 KB, patch)
2011-04-12 16:04 PDT, Lucas Forschler
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
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.