Make building seccomp-bpf a GYP condition

components/nacl.gyp

 # Copyright 2013 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.
 
 {
   'variables': {
     'chromium_code': 1,
   },
   'includes': [
     'nacl/nacl_defines.gypi',
@@ skipping 173 matching lines @@
                 # Allow .cc files to know if they're being compiled as part
                 # of nacl_helper.
                 'IN_NACL_HELPER=1',
               ],
               'sources': [
                 'nacl/loader/nacl_sandbox_linux.cc',
                 'nacl/loader/nacl_helper_linux.cc',
                 'nacl/loader/nacl_helper_linux.h',
                 '../base/posix/unix_domain_socket_linux.cc',
                 '../content/common/child_process_sandbox_support_impl_shm_linux.cc',
-                '../content/common/sandbox_linux/sandbox_bpf_base_policy_linux.cc',
-                '../content/common/sandbox_linux/sandbox_init_linux.cc',
-                '../content/common/sandbox_linux/sandbox_seccomp_bpf_linux.cc',
                 '../content/public/common/content_switches.cc',
               ],
               'conditions': [
                 ['toolkit_uses_gtk == 1', {
                   'dependencies': [
                     '../build/linux/system.gyp:gtk',
                   ],
                 }],
                 ['use_glib == 1', {
                   'dependencies': [
                     '../build/linux/system.gyp:glib',
                   ],
                 }],
                 ['os_posix == 1 and OS != "mac"', {
                   'conditions': [
                     ['linux_use_tcmalloc==1', {
                       'dependencies': [
                         '../base/allocator/allocator.gyp:allocator',
                       ],
                     }],
                   ],
                 }],
+                ['use_seccomp_bpf == 1', {
+                  'sources': [
+                    '../content/common/sandbox_linux/sandbox_bpf_base_policy_linux.cc',
+                    '../content/common/sandbox_linux/sandbox_init_linux.cc',
+                    '../content/common/sandbox_linux/sandbox_seccomp_bpf_linux.cc',
+                  ],
+                  'defines': ['USE_SECCOMP_BPF'],
+                }, {
+                  'sources': [
+                    # Included in both cases

[Mark Seaborn, 2013/12/18 08:10:21]

So why not leave it in the unconditional sources list above?  (With a comment if
you want one.)

[jln (very slow on Chromium), 2013/12/19 00:00:19]

Yeah, I think I misunderstood how "defines" worked.

Done, thanks.

+                    '../content/common/sandbox_linux/sandbox_seccomp_bpf_linux.cc',
+                  ],
+                }],
               ],
               'cflags': ['-fPIE'],
               'link_settings': {
                 'ldflags': ['-pie'],
               },
             },
           ],
         }],
         ['OS=="win" and target_arch=="ia32"', {
           'targets': [
@@ skipping 133 matching lines @@
         'nacl/common/nacl_types.h',
         'nacl/common/pnacl_types.cc',
         'nacl/common/pnacl_types.h',
       ],
       'include_dirs': [
         '..',
       ],
     },
   ]
 }