Non-SFI Mode: Build ipc/ library by PNaCl toolchain for nacl_helper_nonsfi.

components/nacl_nonsfi.gyp

 # Copyright 2014 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': [
     '../build/common_untrusted.gypi',
   ],
   'conditions': [
     ['disable_nacl==0 and disable_nacl_untrusted==0', {
       'targets': [
         {
+          # nacl_helper_nonsfi is similar to nacl_helper (built in nacl.gyp)
+          # but for the NaCl plugin in Non-SFI mode.
+          # This binary is built by the PNaCl toolchain, but it is native

[Mark Seaborn, 2014/10/23 03:33:48]

Subtle nit: "by" -> "using" (since Gyp is involved too)

[hidehiko, 2014/10/23 14:19:08]

Done.

+          # linux binary (translated from a biased pexe) and will run on Linux

[Mark Seaborn, 2014/10/23 03:33:48]

Nit: remove "(translated from a biased pexe)" -- we don't link it as a pexe.

[hidehiko, 2014/10/23 14:19:08]

Done.

+          # directly.
+          # Most library code can be shared with the one for untrusted build
+          # (i.e. the one for irt.nexe built by the NaCl/PNaCl toolchain), but
+          # as nacl_helper_nonsfi runs on Linux, there are some differences,
+          # such as MessageLoopForIO (which is based on libevent in Non-SFI
+          # mode) or ipc_channel implementation.
+          # Because of the toolchain, in both build, OS_NACL macro (derived

[Mark Seaborn, 2014/10/23 03:33:48]

"in both builds" (plural)

[hidehiko, 2014/10/23 14:19:08]

Done.

+          # from __native_client__ macro) is defined. To figure out if it
+          # compiles for SFI-mode or Non-SFI mode, __native_client_nonsfi__

[Mark Seaborn, 2014/10/23 03:33:48]

"if it compiles for" -> "if it is being compiled for"

Maybe reword to avoid the passive voice.  e.g. "Code can test whether
__native_client_nonsfi__ is #defined in order to determine whether it is being
compiled for SFI mode or Non-SFI mode."

[hidehiko, 2014/10/23 14:19:08]

Done. Thank you for suggestion.

+          # macro can be used, which is defined on Non-SFI mode build, but not
+          # on SFI-mode build.
+          #
           # Currently, nacl_helper_nonsfi is under development and the binary
           # does nothing (i.e. it has only empty main(), now).
           # TODO(crbug.com/358465): Implement it then switch nacl_helper in
           # Non-SFI mode to nacl_helper_nonsfi.
           'target_name': 'nacl_helper_nonsfi',
           'type': 'none',
           'variables': {
             'nacl_untrusted_build': 1,
             'nexe_target': 'nacl_helper_nonsfi',
             # Rename the output binary file to nacl_helper_nonsfi and put it
             # directly under out/{Debug,Release}/.
             'out_newlib32_nonsfi': '<(PRODUCT_DIR)/nacl_helper_nonsfi',
 
             'build_glibc': 0,
             'build_newlib': 0,
             'build_irt': 0,
             'build_pnacl_newlib': 0,
             'build_nonsfi_helper': 1,
 
             'sources': [
               'nacl/loader/nacl_helper_linux.cc',
               'nacl/loader/nacl_helper_linux.h',
             ],
 
             'conditions': [
               ['target_arch=="ia32" or target_arch=="x64"', {
                 'extra_deps_newlib32_nonsfi': [
                   '>(tc_lib_dir_nonsfi_helper32)/libbase_nacl_nonsfi.a',
                   '>(tc_lib_dir_nonsfi_helper32)/libevent_nacl_nonsfi.a',
+                  '>(tc_lib_dir_nonsfi_helper32)/libipc_nacl_nonsfi.a',
                   '>(tc_lib_dir_nonsfi_helper32)/libshared_memory_support_nacl.a',
                 ],
               }],
             ],
           },
           'dependencies': [
             '../base/base_nacl.gyp:base_nacl_nonsfi',
+            '../ipc/ipc_nacl.gyp:ipc_nacl_nonsfi',
             '../native_client/src/nonsfi/irt/irt.gyp:nacl_sys_private',
             '../native_client/src/untrusted/nacl/nacl.gyp:nacl_lib_newlib',
             '../native_client/tools.gyp:prep_toolchain',
 
             # Temporarily depends on some libraries to make sure they can be
             # built properly. These are depended on by PPAPI library.
             # TODO(hidehiko): Remove them when PPAPI library is introduced.
             '../media/media_nacl.gyp:shared_memory_support_nacl',
           ],
         },
         # TODO(hidehiko): Add Non-SFI version of nacl_loader_unittests.
       ],
     }],
   ],
 }