Move NetworkService mojom files to content/public/common.

content/public/common/BUILD.gn

Index: content/public/common/BUILD.gn
diff --git a/content/public/common/BUILD.gn b/content/public/common/BUILD.gn
index 0e20d7f514c027c34bd83f2349ceca968207df56..2ce68b81d2a5456287e3d7ece92cd3b3593342f5 100644
--- a/content/public/common/BUILD.gn
+++ b/content/public/common/BUILD.gn
@@ -170,6 +170,7 @@ source_set("common_sources") {
     "mhtml_generation_params.h",
     "mojo_channel_switches.cc",
     "mojo_channel_switches.h",
+    "mutable_network_traffic_annotation_tag_struct_traits.h",
     "notification_resources.cc",
     "notification_resources.h",
     "origin_trial_policy.cc",
@@ -208,6 +209,8 @@ source_set("common_sources") {
     "resource_request.h",
     "resource_request_body.cc",
     "resource_request_body.h",
+    "resource_request_completion_status.cc",
+    "resource_request_completion_status.h",
     "resource_response.cc",
     "resource_response.h",
     "resource_response_info.cc",
@@ -259,9 +262,9 @@ source_set("common_sources") {
   public_configs = [ "//v8:external_startup_data" ]
 
   public_deps = [
+    ":interfaces",
     ":service_names",
     "//content/common",
-    "//content/public/common:interfaces",
     "//ipc",
     "//mojo/edk/system",
     "//mojo/public/cpp/bindings",
@@ -304,6 +307,7 @@ source_set("common_sources") {
 
   # //content/common needs to include public headers.
   allow_circular_includes_from = [
+    ":interfaces",
     "//content/common",
     "//content/common:mojo_bindings",
   ]
@@ -341,10 +345,31 @@ source_set("feature_h264_with_openh264_ffmpeg") {
 }
 
 mojom("interfaces") {

[Nico, 2017/08/16 20:54:39]

Doesn't this now need a public_dep on ui/accessibility? That's what depends on
ax_gen which generates ui/accessibility/ax_enums.h, which is used while this
target is built:

FAILED: obj/content/public/common/interfaces/url_loader_factory.mojom.obj 
../../third_party/llvm-build/Release+Asserts/bin/clang-cl --rsp-quoting=posix
/nologo /showIncludes 
@obj/content/public/common/interfaces/url_loader_factory.mojom.obj.rsp /c
gen/content/public/common/url_loader_factory.mojom.cc
/Foobj/content/public/common/interfaces/url_loader_factory.mojom.obj
/Fd"obj/content/public/common/interfaces_cc.pdb"
In file included from gen/content/public/common/url_loader_factory.mojom.cc:29:
In file included from ../../content/common/resource_messages.h:17:
In file included from ../../content/public/common/common_param_traits.h:24:
In file included from
../../content/public/common/common_param_traits_macros.h:28:
../../ui/accessibility/ax_node_data.h(17,10):  fatal error:
'ui/accessibility/ax_enums.h' file not found
#include "ui/accessibility/ax_enums.h"
         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~

(Or maybe that dependency is missing elsewhere in content?)

[Nico, 2017/08/16 21:02:44]

After some more digging, I think this flake is unrelated to your change;
https://chromium-review.googlesource.com/617768 might fix it.

+  # Must depend on //content/public/common/ instead, for component build.
+  visibility = [
+    ":common_sources",
+    "//content/common/*",
+  ]
+
   sources = [
+    "mutable_network_traffic_annotation_tag.mojom",
+    "network_service.mojom",
     "network_service_test.mojom",
+    "url_loader.mojom",
+    "url_loader_factory.mojom",
     "window_container_type.mojom",
   ]
+
+  public_deps = [
+    "//mojo/common:common_custom_types",
+    "//url/mojo:url_mojom_gurl",
+    "//url/mojo:url_mojom_origin",
+  ]
+
+  component_output_prefix = "content_public_common_mojo_bindings"
+  export_class_attribute = "CONTENT_EXPORT"
+  export_define = "CONTENT_IMPLEMENTATION=1"
+  export_header = "content/common/content_export.h"
 }
 
 mojom("service_names") {