Move Resource Coordinator off Service Manager (reland)
Original CL:
https://chromium-review.googlesource.com/c/chromium/src/+/1769102
PS2 is the original CL rebased. Diff PS2->PS11 to see the one-line fix
for bug 1000327. This is applied in addition to the base change in
https://chromium-review.googlesource.com/c/chromium/src/+/1810036 to fix
a more general race condition in render process host reuse.
This moves the Resource Coordinator Service off of Service Manager APIs
in favor of direct Mojo interface usage. A public browser API is exposed
to give content clients access to the service connection.
NOPRESUBMIT=true
Bug: 977637, 1000327
Change-Id: Id3742584f25b3891386e62789db1b4121b0d3007
TBR: [email protected]
TBR: [email protected]
TBR: [email protected]
TBR: [email protected]
TBR: [email protected]
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1809098
Reviewed-by: François Doray <[email protected]>
Reviewed-by: Ken Rockot <[email protected]>
Commit-Queue: Ken Rockot <[email protected]>
Cr-Commit-Position: refs/heads/master@{#697684}
diff --git a/content/browser/BUILD.gn b/content/browser/BUILD.gn
index 2373983..7955eb7 100644
--- a/content/browser/BUILD.gn
+++ b/content/browser/BUILD.gn
@@ -168,7 +168,6 @@
"//services/network/public/cpp:manifest",
"//services/network/public/mojom",
"//services/resource_coordinator:lib",
- "//services/resource_coordinator/public/cpp:manifest",
"//services/resource_coordinator/public/cpp:resource_coordinator_cpp",
"//services/service_manager",
"//services/service_manager/embedder",
@@ -1602,6 +1601,7 @@
"resolve_proxy_msg_helper.h",
"resource_context_impl.cc",
"resource_context_impl.h",
+ "resource_coordinator_service.cc",
"sandbox_host_linux.cc",
"sandbox_host_linux.h",
"sandbox_ipc_linux.cc",