GN: Build nacl_helper_nonsfi unit tests

base/json/BUILD.gn

 # Copyright (c) 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.
 
+import("//build/config/nacl/config.gni")
+
 source_set("json") {
   sources = [
     "json_file_value_serializer.cc",
     "json_file_value_serializer.h",
     "json_parser.cc",
     "json_parser.h",
     "json_reader.cc",
     "json_reader.h",
     "json_string_value_serializer.cc",
     "json_string_value_serializer.h",
     "json_value_converter.cc",
     "json_value_converter.h",
     "json_writer.cc",
     "json_writer.h",
     "string_escape.cc",
     "string_escape.h",
   ]
 
-  if (is_nacl) {
+  if (is_nacl && !is_nacl_nonsfi) {

[Mark Seaborn, 2015/12/03 19:05:07]

I don't see this being conditionalised in the Gyp build.  If it builds with
newlib, can we remove this "source -= ..." for simplicity?

[Petr Hosek, 2015/12/04 05:57:46]

Unfortunately not, this breaks under newlib because of missing file path support
from base.

     sources -= [
       "json_file_value_serializer.cc",
       "json_file_value_serializer.h",
     ]
   }
 
   configs += [ "//base:base_implementation" ]
 
   deps = [
     "//base/memory",
   ]
 
   visibility = [ "//base/*" ]
 }