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

Unified Diff: content/public/common/sandboxed_process_launcher_delegate.h

Issue 2850903002: Clean up Linux zygote creation code (Closed)
Patch Set: Created 3 years, 8 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 side-by-side diff with in-line comments
Download patch
Index: content/public/common/sandboxed_process_launcher_delegate.h
diff --git a/content/public/common/sandboxed_process_launcher_delegate.h b/content/public/common/sandboxed_process_launcher_delegate.h
index 067274548cd2ed5166b24d5e311f55485d0664a6..56ee5092460117a5bad0891cce56913f1ddf87d8 100644
--- a/content/public/common/sandboxed_process_launcher_delegate.h
+++ b/content/public/common/sandboxed_process_launcher_delegate.h
@@ -50,9 +50,12 @@ class CONTENT_EXPORT SandboxedProcessLauncherDelegate {
virtual void PostSpawnTarget(base::ProcessHandle process) {}
#elif defined(OS_POSIX)
+
#if !defined(OS_MACOSX) && !defined(OS_ANDROID)
- // Override this to return true to use the setuid sandbox.
- virtual ZygoteHandle* GetZygote();
+ // Returns the zygote used to launch the process.
+ // NOTE: For now Chrome always uses the same zygote for performance reasons.
+ // http://crbug.com/569191
+ virtual ZygoteHandle GetZygote();
#endif // !defined(OS_MACOSX) && !defined(OS_ANDROID)
// Override this if the process needs a non-empty environment map.

Powered by Google App Engine
This is Rietveld 408576698