Update BUILD.gn template

BUILD.gn

 # GRPC Chromium BUILD.gn file.
 # This file has been automatically generated from a template file.
 # Please look at the templates directory instead.
 # This file can be regenerated from the template by running
 # tools/buildgen/generate_projects.sh
 config("grpc_config") {
   include_dirs = [
     ".",
     "include/",
   ]
 
   defines = [ "GRPC_USE_PROTO_LITE" ]
 }
 
 config("grpc_no_cxx11_threads") {

[Garrett Casto, 2016/08/15 21:20:59]

Is there any reason why this is separate from "grpc_config" now since we want
both to be public_deps?

[xyzzyz, 2016/08/15 21:59:20]

Merged.

   # TODO(xyzzyz): the <condition_variable> header in libstdc++-4.6 we're using
   # in Chromium has a bug, which causes a compilation error on Clang.
   # Therefore, we need to make gRPC not use standard library threading support.
   # https://crbug.com/593874
   defines = [ "GRPC_CXX0X_NO_THREAD" ]
 }
 
 
 
 
@@ skipping 98 matching lines @@
     "src/core/lib/support/tmpfile_posix.c",
     "src/core/lib/support/tmpfile_windows.c",
     "src/core/lib/support/wrap_memcpy.c",
   ]
   deps = [
   ]
   configs -= [ "//build/config/compiler:chromium_code" ]
   configs += [
     "//build/config/compiler:no_chromium_code",
   ]
-  public_configs = [ ":grpc_config" ]
+  public_configs = [
+    ":grpc_config",
+  ]
 }
 
 
 
 component("grpc") {
   sources = [
     "include/grpc/byte_buffer.h",
     "include/grpc/byte_buffer_reader.h",
     "include/grpc/compression.h",
     "include/grpc/grpc.h",
@@ skipping 356 matching lines @@
   ]
   deps = [
     "//third_party/boringssl",
     "//third_party/zlib",
     ":gpr",
   ]
   configs -= [ "//build/config/compiler:chromium_code" ]
   configs += [
     "//build/config/compiler:no_chromium_code",
   ]
-  public_configs = [ ":grpc_config" ]
+  public_configs = [
+    ":grpc_config",
+  ]
 }
 
 
 
 component("grpc_cronet") {
   sources = [
     "include/grpc/byte_buffer.h",
     "include/grpc/byte_buffer_reader.h",
     "include/grpc/compression.h",
     "include/grpc/grpc.h",
@@ skipping 331 matching lines @@
     "src/core/plugin_registry/grpc_cronet_plugin_registry.c",
   ]
   deps = [
     "//third_party/boringssl",
     ":gpr",
   ]
   configs -= [ "//build/config/compiler:chromium_code" ]
   configs += [
     "//build/config/compiler:no_chromium_code",
   ]
-  public_configs = [ ":grpc_config" ]
+  public_configs = [
+    ":grpc_config",
+  ]
 }
 
 
 
 component("grpc_unsecure") {
   sources = [
     "include/grpc/byte_buffer.h",
     "include/grpc/byte_buffer_reader.h",
     "include/grpc/compression.h",
     "include/grpc/grpc.h",
@@ skipping 293 matching lines @@
     "src/core/ext/census/grpc_filter.c",
     "src/core/ext/census/grpc_plugin.c",
     "src/core/ext/census/initialize.c",
     "src/core/ext/census/mlog.c",
     "src/core/ext/census/operation.c",
     "src/core/ext/census/placeholders.c",
     "src/core/ext/census/tracing.c",
     "src/core/plugin_registry/grpc_unsecure_plugin_registry.c",
   ]
   deps = [
+    "//third_party/zlib",

[Garrett Casto, 2016/08/15 21:20:59]

Can you add a comment in the CL description for this or split this into a
separate CL? Was this causing a compilation error?

[xyzzyz, 2016/08/15 21:59:20]

Yes, it was. Editing CL description.

     ":gpr",
   ]
   configs -= [ "//build/config/compiler:chromium_code" ]
   configs += [
     "//build/config/compiler:no_chromium_code",
   ]
-  public_configs = [ ":grpc_config" ]
+  public_configs = [
+    ":grpc_config",
+  ]
 }
 
 
 
 component("grpc++") {
   sources = [
     "include/grpc++/alarm.h",
     "include/grpc++/channel.h",
     "include/grpc++/client_context.h",
     "include/grpc++/completion_queue.h",
@@ skipping 133 matching lines @@
     "src/cpp/codegen/codegen_init.cc",
   ]
   deps = [
     "//third_party/boringssl",
     "//third_party/protobuf:protobuf_lite",
     ":grpc",
   ]
   configs -= [ "//build/config/compiler:chromium_code" ]
   configs += [
     "//build/config/compiler:no_chromium_code",
+  ]
+  public_configs = [
+    ":grpc_config",
     ":grpc_no_cxx11_threads",
   ]
-  public_configs = [ ":grpc_config" ]
 }
 
 
 
 component("grpc++_unsecure") {
   sources = [
     "include/grpc++/alarm.h",
     "include/grpc++/channel.h",
     "include/grpc++/client_context.h",
     "include/grpc++/completion_queue.h",
@@ skipping 125 matching lines @@
   ]
   deps = [
     "//third_party/protobuf:protobuf_lite",
     ":gpr",
     ":grpc_unsecure",
     ":grpc",
   ]
   configs -= [ "//build/config/compiler:chromium_code" ]
   configs += [
     "//build/config/compiler:no_chromium_code",
+  ]
+  public_configs = [
+    ":grpc_config",
     ":grpc_no_cxx11_threads",
   ]
-  public_configs = [ ":grpc_config" ]
 }
 
 
 
 component("grpc_plugin_support") {
   sources = [
     "include/grpc++/impl/codegen/config_protobuf.h",
     "src/compiler/config.h",
     "src/compiler/cpp_generator.h",
     "src/compiler/cpp_generator_helpers.h",
@@ skipping 16 matching lines @@
     "src/compiler/python_generator.cc",
     "src/compiler/ruby_generator.cc",
   ]
   deps = [
     "//third_party/protobuf:protoc_lib",
   ]
   configs -= [ "//build/config/compiler:chromium_code" ]
   configs += [
     "//build/config/compiler:no_chromium_code",
   ]
-  public_configs = [ ":grpc_config" ]
+  public_configs = [
+    ":grpc_config",
+  ]
 }
 
 
 
 
 executable("grpc_cpp_plugin") {
   sources = [
     "src/compiler/cpp_plugin.cc",
   ]
   deps = [
     "//third_party/protobuf:protoc_lib",
     ":grpc_plugin_support",
   ]
   configs -= [ "//build/config/compiler:chromium_code" ]
   configs += [
     "//build/config/compiler:no_chromium_code",
     "//third_party/protobuf:protobuf_config",
   ]
   public_configs = [ ":grpc_config" ]
 }