Protobuf for Traffic Annotation and first use by a URLFetcher.

net/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/buildflag_header.gni")
 import("//build/config/chromecast_build.gni")
 import("//build/config/compiler/compiler.gni")
 import("//build/config/crypto.gni")
 import("//build/config/features.gni")
 import("//build/config/ui.gni")
@@ skipping 2946 matching lines @@
     "tools/quic/synchronous_host_resolver.h",
   ]
   deps = [
     ":net",
     "//base",
     "//base/third_party/dynamic_annotations",
     "//url",
   ]
 }
 
+source_set("traffic_annotation_header") {

[asanka, 2017/01/26 20:59:27]

why is this in a separate source set when url_fetcher.h and
url_request_context.h depends on it?

[Ramin Halavati, 2017/01/27 07:48:58]

I separated it so that tools that just use annotation definition, like the ones
that analyze annotations, create reports, check policy definitions, etc., won't
need to define a dependency on the whole net. But it can be done the other way
as well if you see that a better choice.

[asanka, 2017/01/27 16:58:00]

Got it.

Can you add :traffic_annotation_header (or rename it to traffic_annotation since
the fact that it's a header isn't important to net) to the |deps| of the source
sets that depend on it?

[Ramin Halavati, 2017/01/30 08:43:25]

Done, renamed it to traffic_annotation and added it "net" component" which was
the only source set having url_fetcher or url_requestion_context.

+  sources = [
+    "net/traffic_annotation/network_traffic_annotation.h",
+  ]
+  deps = []
+}
+
 if (!is_ios) {
   executable("quic_client") {
     sources = [
       "tools/quic/quic_simple_client_bin.cc",
     ]
     deps = [
       ":net",
       ":simple_quic_tools",
       "//base",
       "//build/config/sanitizers:deps",
@@ skipping 2667 matching lines @@
       "tools/domain_security_preload_generator/trie/trie_writer.cc",
       "tools/domain_security_preload_generator/trie/trie_writer.h",
     ]
     deps = [
       "//base",
       "//crypto",
       "//third_party/boringssl",
     ]
   }
 }