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

Side by Side Diff: content/common/process_watcher_posix.cc

Issue 7477008: Remove explicit keyword from multi-argument constructors (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 years, 5 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « content/browser/zygote_main_linux.cc ('k') | gpu/command_buffer/service/gles2_cmd_decoder.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2009 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2009 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/common/process_watcher.h" 5 #include "content/common/process_watcher.h"
6 6
7 #include <errno.h> 7 #include <errno.h>
8 #include <signal.h> 8 #include <signal.h>
9 #include <sys/types.h> 9 #include <sys/types.h>
10 #include <sys/wait.h> 10 #include <sys/wait.h>
(...skipping 14 matching lines...) Expand all
25 return true; 25 return true;
26 } 26 }
27 27
28 return false; 28 return false;
29 } 29 }
30 30
31 // A thread class which waits for the given child to exit and reaps it. 31 // A thread class which waits for the given child to exit and reaps it.
32 // If the child doesn't exit within a couple of seconds, kill it. 32 // If the child doesn't exit within a couple of seconds, kill it.
33 class BackgroundReaper : public base::PlatformThread::Delegate { 33 class BackgroundReaper : public base::PlatformThread::Delegate {
34 public: 34 public:
35 explicit BackgroundReaper(pid_t child, unsigned timeout) 35 BackgroundReaper(pid_t child, unsigned timeout)
36 : child_(child), 36 : child_(child),
37 timeout_(timeout) { 37 timeout_(timeout) {
38 } 38 }
39 39
40 void ThreadMain() { 40 void ThreadMain() {
41 WaitForChildToDie(); 41 WaitForChildToDie();
42 delete this; 42 delete this;
43 } 43 }
44 44
45 void WaitForChildToDie() { 45 void WaitForChildToDie() {
(...skipping 50 matching lines...) Expand 10 before | Expand all | Expand 10 after
96 96
97 // static 97 // static
98 void ProcessWatcher::EnsureProcessGetsReaped(base::ProcessHandle process) { 98 void ProcessWatcher::EnsureProcessGetsReaped(base::ProcessHandle process) {
99 // If the child is already dead, then there's nothing to do. 99 // If the child is already dead, then there's nothing to do.
100 if (IsChildDead(process)) 100 if (IsChildDead(process))
101 return; 101 return;
102 102
103 BackgroundReaper* reaper = new BackgroundReaper(process, 0); 103 BackgroundReaper* reaper = new BackgroundReaper(process, 0);
104 base::PlatformThread::CreateNonJoinable(0, reaper); 104 base::PlatformThread::CreateNonJoinable(0, reaper);
105 } 105 }
OLDNEW
« no previous file with comments | « content/browser/zygote_main_linux.cc ('k') | gpu/command_buffer/service/gles2_cmd_decoder.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698