diff options
| author | jeremy@chromium.org <jeremy@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-12-15 11:55:08 +0000 |
|---|---|---|
| committer | jeremy@chromium.org <jeremy@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-12-15 11:55:08 +0000 |
| commit | e36e86f9cf568e12a47cc779893580a09b08803a (patch) | |
| tree | babbda83fb63c64565309a8fd19782f1145c72ce /chrome/browser/child_process_launcher.cc | |
| parent | 86e19d84402a76c57fce85655c24f547312b6084 (diff) | |
| download | chromium_src-e36e86f9cf568e12a47cc779893580a09b08803a.zip chromium_src-e36e86f9cf568e12a47cc779893580a09b08803a.tar.gz chromium_src-e36e86f9cf568e12a47cc779893580a09b08803a.tar.bz2 | |
Make ProcessWatcher use kqueues on Mac.
* Port ProcessWatcher::EnsureProcessTerminated() to kqueue() APIs on OS X.
* Make ProcessWatcher::EnsureProcessGetsReaped() Linux-only, since it's only used there.
* Add a unit test.
BUG=12731
TEST=Open Chrome/Mac, open and close a few tabs. Processes shouldn't stay around.
Review URL: http://codereview.chromium.org/496007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@34547 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/child_process_launcher.cc')
| -rw-r--r-- | chrome/browser/child_process_launcher.cc | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/chrome/browser/child_process_launcher.cc b/chrome/browser/child_process_launcher.cc index 956f836..1ab66c6 100644 --- a/chrome/browser/child_process_launcher.cc +++ b/chrome/browser/child_process_launcher.cc @@ -228,11 +228,11 @@ class ChildProcessLauncher::Context // through the zygote process. Singleton<ZygoteHost>()->EnsureProcessTerminated(handle); } else -#endif // defined(OS_LINUX) +#endif // OS_LINUX { ProcessWatcher::EnsureProcessTerminated(handle); } -#endif +#endif // OS_POSIX process.Close(); } |