Linux sandbox: change the return type of HandleRequest

sandbox/linux/syscall_broker/broker_process.h

Index: sandbox/linux/syscall_broker/broker_process.h
diff --git a/sandbox/linux/syscall_broker/broker_process.h b/sandbox/linux/syscall_broker/broker_process.h
index d5b521b996c622801eb6f29f760c32bd4617cbb9..8d22a9e3d5282075174153653aa937b1915c1e89 100644
--- a/sandbox/linux/syscall_broker/broker_process.h
+++ b/sandbox/linux/syscall_broker/broker_process.h
@@ -19,8 +19,8 @@
 namespace sandbox {
 
 namespace syscall_broker {
+
 class BrokerClient;
-}
 
 // Create a new "broker" process to which we can send requests via an IPC
 // channel by forking the current process.
@@ -68,6 +68,11 @@ class SANDBOX_EXPORT BrokerProcess {
   int broker_pid() const { return broker_pid_; }
 
  private:
+  friend class BrokerProcessTestHelper;
+  // Close the IPC channel with the other party. This should only be used

[mdempsky, 2014/11/10 08:36:35]

nit: I would put a blank line between the friend declaration and here.

[jln (very slow on Chromium), 2014/11/10 08:55:50]

Done.

+  // by tests.
+  void CloseChannel();
+
   bool initialized_;  // Whether we've been through Init() yet.
   bool is_child_;     // Whether we're the child (broker process).
   bool fast_check_in_client_;
@@ -78,11 +83,12 @@ class SANDBOX_EXPORT BrokerProcess {
       broker_client_;  // Can only exist if is_child_ is true.
 
   int ipc_socketpair_;  // Our communication channel to parent or child.
-  DISALLOW_COPY_AND_ASSIGN(BrokerProcess);
 
-  friend class BrokerProcessTestHelper;
+  DISALLOW_COPY_AND_ASSIGN(BrokerProcess);
 };
 
+}  // namespace syscall_broker
+
 }  // namespace sandbox
 
 #endif  // SANDBOX_LINUX_SERVICES_BROKER_PROCESS_H_