Do not use O_CLOEXEC for open() in the broker process.

sandbox/linux/services/broker_process_unittest.cc

Index: sandbox/linux/services/broker_process_unittest.cc
diff --git a/sandbox/linux/services/broker_process_unittest.cc b/sandbox/linux/services/broker_process_unittest.cc
index c0393ed2f50bac2dcacbbdd6f30f64e2014a9086..8ff4e00c8eca585f6d0ffcbeea78184f6261cdfb 100644
--- a/sandbox/linux/services/broker_process_unittest.cc
+++ b/sandbox/linux/services/broker_process_unittest.cc
@@ -405,14 +405,6 @@ void TestOpenComplexFlags(bool fast_check_in_client) {
   // Important: use F_GETFD, not F_GETFL. The O_CLOEXEC flag in F_GETFL
   // is actually not used by the kernel.
   ASSERT_TRUE(FD_CLOEXEC & ret);
-
-  // There is buggy userland code that can check for O_CLOEXEC with fcntl(2)
-  // even though it doesn't mean anything. We need to support this case.
-  // See crbug.com/237283.
-  ret = fcntl(fd, F_GETFL);
-  ASSERT_NE(-1, ret);
-  ASSERT_TRUE(O_CLOEXEC & ret);
-
   ASSERT_EQ(0, close(fd));
 
   fd = open_broker.Open(kCpuInfo, O_RDONLY | O_NONBLOCK);