Non-SFI mode: Enable browser_tests for nacl_helper_nonsfi.

chrome/test/nacl/nacl_browsertest_util.h

Index: chrome/test/nacl/nacl_browsertest_util.h
diff --git a/chrome/test/nacl/nacl_browsertest_util.h b/chrome/test/nacl/nacl_browsertest_util.h
index ac084d0aeb99ce210584c1442e2088ee05a9b371..86d8e62881c493f3651c5ea2ee4165f4bd945165 100644
--- a/chrome/test/nacl/nacl_browsertest_util.h
+++ b/chrome/test/nacl/nacl_browsertest_util.h
@@ -129,12 +129,23 @@ class NaClBrowserTestPnaclNonSfi : public NaClBrowserTestBase {
   base::FilePath::StringType Variant() override;
 };
 
+class NaClBrowserTestPnaclNaClHelperNonSfi
+    : public NaClBrowserTestPnaclNonSfi {
+ public:
+  void SetUpCommandLine(base::CommandLine* command_line) override;
+};
+
 class NaClBrowserTestNonSfiMode : public NaClBrowserTestBase {
  public:
   void SetUpCommandLine(base::CommandLine* command_line) override;
   base::FilePath::StringType Variant() override;
 };
 
+class NaClBrowserTestNaClHelperNonSfi : public NaClBrowserTestNonSfiMode {
+ public:
+  void SetUpCommandLine(base::CommandLine* command_line) override;
+};
+
 // A NaCl browser test only using static files.
 class NaClBrowserTestStatic : public NaClBrowserTestBase {
  public:
@@ -181,6 +192,16 @@ class NaClBrowserTestGLibcExtension : public NaClBrowserTestGLibc {
 #  define MAYBE_NONSFI(test_case) DISABLED_##test_case
 #endif
 
+// Currently, we only supports it on x86-32 architecture.
+// TODO(hidehiko,mazda): Enable this on ARM, too, when it is supported.
+#if defined(OS_LINUX) && !defined(ADDRESS_SANITIZER) && \
+    !defined(THREAD_SANITIZER) && !defined(MEMORY_SANITIZER) && \
+  !defined(LEAK_SANITIZER) && defined(ARCH_CPU_X86)
+#  define MAYBE_NACL_HELPER_NONSFI(test_case) test_case
+#else
+#  define MAYBE_NACL_HELPER_NONSFI(test_case) DISABLED_##test_case
+#endif
+
 // Currently, translation from pexe to non-sfi nexe is supported only for
 // x86-32 or ARM binary.
 #if defined(OS_LINUX) && (defined(ARCH_CPU_X86) || defined(ARCH_CPU_ARMEL))
@@ -189,6 +210,16 @@ class NaClBrowserTestGLibcExtension : public NaClBrowserTestGLibc {
 #  define MAYBE_PNACL_NONSFI(test_case) DISABLED_##test_case
 #endif
 
+// Similar to MAYBE_NACL_HELPER_NONSFI, this is not available on ARM yet.
+// TODO(hidehiko,mazda): Merge this to the MAYBE_PNACL_NONSFI when it is
+// supported on ARM.
+#if defined(OS_LINUX) && defined(ARCH_CPU_X86)

[Mark Seaborn, 2014/11/17 18:35:19]

Shouldn't this be ARCH_CPU_X86_FAMILY so that these tests get enabled on the
x86-64 bots too?

(Plus add a comment to say why the old and new versions differ in this respect.)

[hidehiko, 2014/11/18 04:39:37]

Non-SFI testee binaries are built only for ia32 arch.

[Mark Seaborn, 2014/11/19 06:23:54]

OK.  I assume you'll fix this in a later change?

+#  define MAYBE_PNACL_NACL_HELPER_NONSFI(test_case) test_case
+#else
+#  define MAYBE_PNACL_NACL_HELPER_NONSFI(test_case) DISABLED_##test_case
+#endif
+
+
 #define NACL_BROWSER_TEST_F(suite, name, body) \
 IN_PROC_BROWSER_TEST_F(suite##Newlib, name) \
 body \