Clean up Linux zygote creation code

content/public/common/sandboxed_process_launcher_delegate.cc

Index: content/public/common/sandboxed_process_launcher_delegate.cc
diff --git a/content/public/common/sandboxed_process_launcher_delegate.cc b/content/public/common/sandboxed_process_launcher_delegate.cc
index 5435816be6d268f9a285b5b4ea3fcb4a99c07b84..84813c5e915e67316a43a3c050a0930fc6ec35a7 100644
--- a/content/public/common/sandboxed_process_launcher_delegate.cc
+++ b/content/public/common/sandboxed_process_launcher_delegate.cc
@@ -2,9 +2,10 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#include "build/build_config.h"
 #include "content/public/common/sandboxed_process_launcher_delegate.h"
 
+#include "build/build_config.h"
+
 namespace content {
 
 #if defined(OS_WIN)
@@ -26,8 +27,9 @@ bool SandboxedProcessLauncherDelegate::PreSpawnTarget(
 }
 
 #elif(OS_POSIX)
+
 #if !defined(OS_MACOSX) && !defined(OS_ANDROID)
-ZygoteHandle* SandboxedProcessLauncherDelegate::GetZygote() {
+ZygoteHandle SandboxedProcessLauncherDelegate::GetZygote() {
   return nullptr;

[James Cook, 2017/04/28 20:07:52]

I could also make this return GetGenericZygote().

[mdempsky, 2017/04/28 22:13:45]

I believe some of the launcher delegate subclasses do not want to use zygotes,
so you'd have to override GetZygote in those to return nullptr instead. I'd
suggest leaving as is.

 }
 #endif  // !defined(OS_MACOSX) && !defined(OS_ANDROID)