New NaCl zygote implementation 2

chrome/common/nacl_fork_delegate_linux.h

+// Copyright (c) 2011 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef CHROME_COMMON_NACL_FORK_DELEGATE_LINUX_H_
+#define CHROME_COMMON_NACL_FORK_DELEGATE_LINUX_H_
+#pragma once
+
+#include "base/basictypes.h"
+#include "base/compiler_specific.h"
+#include "content/common/zygote_fork_delegate_linux.h"
+
+// The NaClForkDelegate is created during Chrome linux zygote
+// initialization, and provides "fork()" functionality with
+// NaCl specific process characteristics (specifically address
+// space layout) as an alternative to forking the zygote.
+// A new delegate is passed in as an argument to ZygoteMain().
+class NaClForkDelegate : public ZygoteForkDelegate {
+ public:
+  NaClForkDelegate();
+  ~NaClForkDelegate();
+
+  virtual void Init(const bool sandboxed,

[Evan Martin, 2011/06/27 18:49:32]

nit: I think one of our compilers (clang?) will warn about the meaningless
"const" applied to a param that is passed by value.  (here, below on the "int"s
as well)

I bet if you land this you can just fix it once one of the bots go red, though,
if you prefer.

[Brad Chen, 2011/06/27 22:06:25]

Done.

+                    const int browserdesc,
+                    const int sandboxdesc) OVERRIDE;
+  virtual bool CanHelp(const std::string& process_type) OVERRIDE;
+  virtual pid_t Fork(const std::vector<int>& fds) OVERRIDE;
+  virtual bool AckChild(const int fd,
+                        const std::string& channel_switch) OVERRIDE;
+
+ private:
+  bool sandboxed_;
+  int fd_;
+  pid_t pid_;
+};
+
+#endif  // CHROME_COMMON_NACL_FORK_DELEGATE_LINUX_H_