Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(335)

Unified Diff: tools/task_kill.py

Issue 131743002: Add content_shell to the list of processes that get killed in the task_kill.py step (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 6 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/task_kill.py
diff --git a/tools/task_kill.py b/tools/task_kill.py
index ec6b574b6077d807386cfe76799013d15ac8175b..fa7bbd51e31dfc25f2de3c6e494220c4135bb15b 100755
--- a/tools/task_kill.py
+++ b/tools/task_kill.py
@@ -23,12 +23,26 @@ os_name = utils.GuessOS()
POSIX_INFO = 'ps -p %s -o args'
EXECUTABLE_NAMES = {
- 'win32': { 'chrome': 'chrome.exe', 'dart': 'dart.exe',
- 'iexplore': 'iexplore.exe', 'firefox': 'firefox.exe'},
- 'linux': { 'chrome': 'chrome', 'dart': 'dart',
- 'firefox': 'firefox.exe'},
- 'macos': { 'chrome': 'Chrome', 'dart': 'dart',
- 'firefox': 'firefox', 'safari': 'Safari' }
+ 'win32': {
+ 'chrome': 'chrome.exe',
+ 'content_shell': 'content_shell.exe',
+ 'dart': 'dart.exe',
+ 'iexplore': 'iexplore.exe',
+ 'firefox': 'firefox.exe'
+ },
+ 'linux': {
+ 'chrome': 'chrome',
+ 'content_shell': 'content_shell',
+ 'dart': 'dart',
+ 'firefox': 'firefox.exe'
+ },
+ 'macos': {
+ 'chrome': 'Chrome',
+ 'content_shell': 'Content Shell',
+ 'dart': 'dart',
+ 'firefox': 'firefox',
+ 'safari': 'Safari'
+ }
}
INFO_COMMAND = {
@@ -153,6 +167,7 @@ def KillBrowsers():
status += Kill('chrome')
status += Kill('iexplore')
status += Kill('safari')
+ status += Kill('content_shell')
return status
def KillDart():
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698