Standardize "net" category trace events

net/BUILD.gn

Index: net/BUILD.gn
diff --git a/net/BUILD.gn b/net/BUILD.gn
index 5f478187ff98d0dcd0ea0cd68109440888920ab5..f60ec01f2d2cc20d417a9341c39266b13868b3d5 100644
--- a/net/BUILD.gn
+++ b/net/BUILD.gn
@@ -86,11 +86,22 @@ if (is_linux) {
   net_configs += [ "//build/config/linux:libresolv" ]
 }
 
+source_set("constants") {

[xunjieli, 2016/12/02 22:35:53]

Thanks so much for the ideas! I was able to reproduce this on my windows
machine. If I try to use net/base/net_errors.h in proxy_resolver_v8_tracing.cc,
I will get the same linker error.

I searched on chromium-dev, it seems that the way to solve this in GN is to have
a separate target either as a source_set or static_library for the
constants(e.g. common:constants, chrome_elf:constants,
remoting/host/mac:constants).

I have updated the CL. Please take a look at the diff between PS#4 and PS#5.

+  sources = [
+    "base/trace_constants.cc",
+    "base/trace_constants.h",
+  ]
+  deps = [
+    "//base",
+  ]
+}
+
 component("net") {
   sources = gypi_values.net_nacl_common_sources
   net_unfiltered_sources = []
 
   deps = [
+    ":constants",
     ":net_resources",
     "//base",
     "//net/base/registry_controlled_domains",
@@ -687,6 +698,7 @@ if (use_v8_in_net) {
     ]
 
     public_deps = [
+      ":constants",
       ":net",
     ]
     deps = [