Turn //mojo/public/cpp/bindings and //mojo/public/cpp/system into components

mojo/public/cpp/bindings/BUILD.gn

 # 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.
 
-static_library("bindings") {
+interfaces_bindings_gen_dir = "$root_gen_dir/mojo/public/interfaces/bindings"
+
+component("bindings") {
   sources = [
+    # Normally, targets should depend on the source_sets generated by mojom
+    # targets. However, the generated source_sets use portions of the bindings
+    # library. In order to avoid linker warnings about locally-defined imports
+    # in Windows components build, this target depends on the generated .cc

[yzshen1, 2016/09/22 22:31:15]

nit: please consider changing ".cc" to "C++", because we have both .cc and .h
files here. 

[dcheng, 2016/09/24 01:06:44]

Done.

+    # files directly so that the EXPORT macro defintions match.
+    "$interfaces_bindings_gen_dir/interface_control_messages.mojom-shared-internal.h",
+    "$interfaces_bindings_gen_dir/interface_control_messages.mojom-shared.cc",
+    "$interfaces_bindings_gen_dir/interface_control_messages.mojom-shared.h",
+    "$interfaces_bindings_gen_dir/interface_control_messages.mojom.cc",
+    "$interfaces_bindings_gen_dir/interface_control_messages.mojom.h",
+    "$interfaces_bindings_gen_dir/pipe_control_messages.mojom-shared-internal.h",
+    "$interfaces_bindings_gen_dir/pipe_control_messages.mojom-shared.cc",
+    "$interfaces_bindings_gen_dir/pipe_control_messages.mojom-shared.h",
+    "$interfaces_bindings_gen_dir/pipe_control_messages.mojom.cc",
+    "$interfaces_bindings_gen_dir/pipe_control_messages.mojom.h",
     "array.h",
     "array_data_view.h",
     "array_traits.h",
     "array_traits_carray.h",
     "array_traits_standard.h",
     "array_traits_stl.h",
     "associated_binding.h",
     "associated_binding_set.h",
     "associated_group.h",
     "associated_group_controller.h",
     "associated_interface_ptr.h",
     "associated_interface_ptr_info.h",
     "associated_interface_request.h",
     "binding.h",
     "binding_set.h",
+    "bindings_export.h",
     "connection_error_callback.h",
     "connector.h",
     "enum_traits.h",
     "filter_chain.h",
     "interface_data_view.h",
     "interface_endpoint_client.h",
     "interface_endpoint_controller.h",
     "interface_id.h",
     "interface_ptr.h",
     "interface_ptr_info.h",
@@ skipping 99 matching lines @@
 
   public_deps = [
     ":struct_traits",
     "//base",
     "//ipc:param_traits",
     "//mojo/public/cpp/system",
   ]
 
   deps = [
     "//base",
-    "//mojo/public/interfaces/bindings:bindings_cpp_sources",
+    "//mojo/public/interfaces/bindings:bindings__generator",
+    "//mojo/public/interfaces/bindings:bindings_shared__generator",
   ]
+
+  defines = [ "MOJO_CPP_BINDINGS_IMPLEMENTATION" ]
 }
 
 source_set("struct_traits") {
   sources = [
     "struct_traits.h",
   ]
 }
 
 if (!is_ios) {
   # TODO(yzshen): crbug.com/617718 Consider moving this into blink.
@@ skipping 12 matching lines @@
     ]
 
     public_deps = [
       ":bindings",
       "//third_party/WebKit/Source/wtf",
     ]
 
     public_configs = [ "//third_party/WebKit/Source:config" ]
   }
 }