Linux sandbox: Make the test run on Android as Native executable

sandbox/linux/services/broker_process_unittest.cc

 // 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.
 
 #include "sandbox/linux/services/broker_process.h"
 
 #include <errno.h>
 #include <fcntl.h>
 #include <sys/stat.h>
 #include <sys/types.h>
 #include <sys/wait.h>
 #include <string>
 #include <vector>
 
 #if defined(OS_ANDROID)
 #include "base/android/path_utils.h"
 #endif
 #include "base/file_path.h"
 #include "base/logging.h"
 #include "sandbox/linux/tests/unit_tests.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace sandbox {
 
+#if defined(OS_ANDROID)
+  #define DISABLE_ON_ANDROID(function) DISABLED_##function
+#else
+  #define DISABLE_ON_ANDROID(function) function
+#endif
+
 TEST(BrokerProcess, CreateAndDestroy) {
   std::vector<std::string> read_whitelist;
   read_whitelist.push_back("/proc/cpuinfo");
 
   BrokerProcess* open_broker = new BrokerProcess(read_whitelist,
                                                  std::vector<std::string>());
   ASSERT_TRUE(open_broker->Init(NULL));
   pid_t broker_pid = open_broker->broker_pid();
   delete(open_broker);
 
@@ skipping 146 matching lines @@
   // Don't do anything here, so that ASSERT works in the subfunction as
   // expected.
 }
 
 TEST(BrokerProcess, OpenCpuinfoNoClientCheck) {
   TestOpenCpuinfo(false /* fast_check_in_client */);
   // Don't do anything here, so that ASSERT works in the subfunction as
   // expected.
 }
 
-TEST(BrokerProcess, OpenFileRW) {
+// Disabled until we implement a mkstemp that doesn't require JNI.
+TEST(BrokerProcess, DISABLE_ON_ANDROID(OpenFileRW)) {
   const char basename[] = "BrokerProcessXXXXXX";
   char template_name[2048];
 #if defined(OS_ANDROID)
   FilePath cache_directory;
   ASSERT_TRUE(base::android::GetCacheDirectory(&cache_directory));
   ssize_t length = snprintf(template_name, sizeof(template_name),
                             "%s%s",
                             cache_directory.value().c_str(), basename);
   ASSERT_LT(length, static_cast<ssize_t>(sizeof(template_name)));
 #else
@@ skipping 93 matching lines @@
   // expected.
 }
 
 TEST(BrokerProcess, ComplexFlagsNoClientCheck) {
   TestComplexFlags(false /* fast_check_in_client */);
   // Don't do anything here, so that ASSERT works in the subfunction as
   // expected.
 }
 
 }  // namespace sandbox