Setuid sandbox client class readability meta-CL

content/common/sandbox_linux.h

 // Copyright (c) 2012 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 CONTENT_COMMON_SANDBOX_LINUX_H_
 #define CONTENT_COMMON_SANDBOX_LINUX_H_
 
 #include <string>
 
 #include "base/basictypes.h"
@@ skipping 27 matching lines @@
 
   // Do some initialization that can only be done before any of the sandboxes
   // is enabled.
   //
   // There are two versions of this function. One takes a process_type
   // as an argument, the other doesn't.
   // It may be necessary to call PreinitializeSandboxBegin before knowing the
   // process type (this is for instance the case with the Zygote).
   // In that case, it is crucial that PreinitializeSandboxFinish() gets
   // called for every child process.
-  // TODO(markus,jln) we know this is not always done at the moment
+  // TODO(markus, jln) we know this is not always done at the moment
   // (crbug.com/139877).
   void PreinitializeSandbox(const std::string& process_type);
   // These should be called together.
   void PreinitializeSandboxBegin();
   void PreinitializeSandboxFinish(const std::string& process_type);
 
   // Returns the Status of the sandbox. Can only be queried if we went through
   // PreinitializeSandbox() or PreinitializeSandboxBegin(). This is a bitmask
   // and uses the constants defined in "enum LinuxSandboxStatus".
   // Since we need to provide the status before the sandboxes are actually
@@ skipping 29 matching lines @@
   scoped_ptr<sandbox::SetuidSandboxClient> setuid_sandbox_client_;
 
   ~LinuxSandbox();
   DISALLOW_IMPLICIT_CONSTRUCTORS(LinuxSandbox);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_COMMON_SANDBOX_LINUX_H_