GN: Build nacl_helper_nonsfi unit tests

sandbox/linux/BUILD.gn

Index: sandbox/linux/BUILD.gn
diff --git a/sandbox/linux/BUILD.gn b/sandbox/linux/BUILD.gn
index 516342513bccb9387c5235f02a728a8e48dd3e41..b3851be4e3c3d4e70ce0a40044536002e94ac402 100644
--- a/sandbox/linux/BUILD.gn
+++ b/sandbox/linux/BUILD.gn
@@ -65,7 +65,7 @@ source_set("sandbox_linux_test_utils") {
     "//testing/gtest",
   ]
 
-  if (use_seccomp_bpf) {
+  if (use_seccomp_bpf || is_nacl_nonsfi) {

[jln (very slow on Chromium), 2015/12/08 23:36:50]

nit: it would be better if is_nacl_nonsfi would imply use_seccomp_bpf instead.
No big deal and no need to do that in this CL.

[Petr Hosek, 2015/12/09 00:51:28]

OK, I'd rather do that as a separate change once we have tests in place.

     sources += [
       "seccomp-bpf/bpf_tester_compatibility_delegate.h",
       "seccomp-bpf/bpf_tests.h",
@@ -79,6 +79,13 @@ source_set("sandbox_linux_test_utils") {
     deps += [ "//base/test:test_support" ]
     defines = [ "SANDBOX_USES_BASE_TEST_SUITE" ]
   }
+
+  if (is_nacl_nonsfi) {
+    sources -= [

[jln (very slow on Chromium), 2015/12/08 23:36:50]

I'm a bit alergic to removing files, it makes things hard to understand. Do you
mind doing a conditional += above instead?

[Petr Hosek, 2015/12/09 00:51:28]

Done.

+      "tests/test_utils.cc",
+      "tests/test_utils.h",
+    ]
+  }
 }
 
 # Sources for sandbox_linux_unittests.