Handled nullptr argument in WaitForExit() and WaitForExitWithTimeout() methods.

base/process/process.h

Index: base/process/process.h
diff --git a/base/process/process.h b/base/process/process.h
index 808baeb1fc6ebdf7a9603b7b9b8d070b37694cdd..114a694abf11b8efb194e83f7d5c4fe6ea172d93 100644
--- a/base/process/process.h
+++ b/base/process/process.h
@@ -102,10 +102,10 @@ class BASE_EXPORT Process {
   // On POSIX, if the process has been signaled then |exit_code| is set to -1.
   // On Linux this must be a child process, however on Mac and Windows it can be
   // any process.
-  bool WaitForExit(int* exit_code);
+  bool WaitForExit(int* exit_code = NULL);

[Lei Zhang, 2015/04/29 07:52:32]

This is against the style guide.
https://google-styleguide.googlecode.com/svn/trunk/cppguide.html#Default_Argu...

You may want to just wait for rvargas to get back to work out what to do here.

[rvargas (doing something else), 2015/04/30 23:28:26]

Right. I didn't mean to have a default argument, just to accept nullptr as a
valid input.

It's going to be a while before I go back so it's better not to wait for me :). 

[g.mehndiratt, 2015/05/04 06:40:21]

On 2015/04/30 23:28:26, rvargas (out of office) wrote:

I will corporate comments.

If u r not available can u tell whom to add for review?

 
   // Same as WaitForExit() but only waits for up to |timeout|.
-  bool WaitForExitWithTimeout(TimeDelta timeout, int* exit_code);
+  bool WaitForExitWithTimeout(TimeDelta timeout, int* exit_code = NULL);
 
   // A process is backgrounded when it's priority is lower than normal.
   // Return true if this process is backgrounded, false otherwise.