Chromium Code Reviews| Index: base/process/launch_posix.cc |
| diff --git a/base/process/launch_posix.cc b/base/process/launch_posix.cc |
| index 99582f59cfcfa760b1459b987e962c56d8a76c9d..5d3fbeee115740b796e581103a19710a1e8e506a 100644 |
| --- a/base/process/launch_posix.cc |
| +++ b/base/process/launch_posix.cc |
| @@ -673,19 +673,6 @@ bool GetAppOutputAndError(const CommandLine& cl, std::string* output) { |
| return result == EXECUTE_SUCCESS && exit_code == EXIT_SUCCESS; |
| } |
| -// TODO(viettrungluu): Conceivably, we should have a timeout as well, so we |
| -// don't hang if what we're calling hangs. |
| -bool GetAppOutputRestricted(const CommandLine& cl, |
| - std::string* output, size_t max_output) { |
| - // Run |execve()| with the empty environment. |
| - char* const empty_environ = NULL; |
| - int exit_code; |
| - GetAppOutputInternalResult result = GetAppOutputInternal( |
|
Lei Zhang
2016/07/01 06:41:48
All the callers to GetAppOutputInternal() now pass
benwells
2016/07/04 03:07:53
Good catch. Removed now.
|
| - cl.argv(), &empty_environ, false, output, max_output, false, &exit_code); |
| - return result == GOT_MAX_OUTPUT || (result == EXECUTE_SUCCESS && |
| - exit_code == EXIT_SUCCESS); |
| -} |
| - |
| bool GetAppOutputWithExitCode(const CommandLine& cl, |
| std::string* output, |
| int* exit_code) { |