Adding nonsfi content handler

services/nacl/BUILD.gn

 # Copyright 2015 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.
 
 import("//mojo/public/mojo_application.gni")
 
 # Trusted code
 if (!is_nacl) {
   mojo_native_application("nacl_content_handler") {
     sources = [
       "content_handler_main.cc",
     ]
 
     deps = [
       "//base",
       "//mojo/application:application",
       "//mojo/application:content_handler",
       "//mojo/data_pipe_utils",
       "//mojo/message_pump",
       "//mojo/public/platform/native:system_impl_private",
       "//mojo/services/network/public/interfaces",
       "//nacl_bindings:monacl_sel",
       "//url",
     ]
 
     data_deps = [ "//nacl_bindings:irt_mojo(//native_client/build/toolchain/nacl:irt_${target_cpu})" ]
   }
+
+  mojo_native_application("nacl_nonsfi_content_handler") {
+    sources = [
+      "nonsfi_content_handler_main.cc",

[Mark Seaborn, 2015/09/01 15:55:45]

Nit: "monacl_shell_nonsfi.cc" is named with "_nonsfi" as a suffix.  How about
using a suffix here too for consistency?

(That's assuming we keep monacl_shell_nonsfi.cc.)

[Sean Klein, 2015/09/01 20:24:47]

Done.

+    ]
+
+    deps = [
+      "//base",
+      "//mojo/application:application",
+      "//mojo/application:content_handler",
+      "//mojo/data_pipe_utils",
+      "//mojo/message_pump",
+      "//mojo/public/platform/nacl:mojo_irt_header",
+      "//mojo/public/platform/native:system_impl_private",
+      "//nacl_bindings:monacl_sel",

[Mark Seaborn, 2015/09/01 15:55:45]

Shouldn't be needed: this refers to SFI NaCl.

[Sean Klein, 2015/09/01 20:24:47]

Done (it was needed for "mojo::NaClExit", which I am no longer using anyway).

+      "//native_client/src/nonsfi/irt:irt_interfaces",
+      "//native_client/src/nonsfi/loader:elf_loader",
+    ]
+  }
 }