Servicifying net::MojoProxyResolverFactory

Desktop / Chromium - Jay Civelli [google.com] - 9 October 2017 16:22 EDT

Making net::MojoProxyResolverFactory a service and removing its use of UtilityProcessHost.

Also renaming the interface implementations from MojoProxyResolverImpl/ MojoProxyResolverFactoryImpl to ProxyResolverImpl/ ProxyResolverFactoryImpl as is customary.

Also moving files not needed by the proxy service from services/proxy_service to content/: proxy_resolver_factory_mojo.[h,cc] proxy_service_mojo.[h,cc] test_mojo_proxy_resolver_factory.[h,cc] mojo_proxy_resolver_factory.h

Bug: 758355 Change-Id: I106f2f3979d41db2ceed554991591653c900ab46 Reviewed-on: https://chromium-review.googlesource.com/651340

2df2464 Servicifying net::MojoProxyResolverFactory.
chrome/app/BUILD.gn | 1 +
chrome/browser/BUILD.gn | 1 +
chrome/browser/DEPS | 3 +
chrome/browser/chrome_content_browser_client.cc | 24 +-
.../chrome_content_browser_manifest_overlay.json | 1 +
chrome/browser/net/DEPS | 1 -
.../net/chrome_mojo_proxy_resolver_factory.cc | 87 ++++----
.../net/chrome_mojo_proxy_resolver_factory.h | 36 +--
...rome_mojo_proxy_resolver_factory_browsertest.cc | 247 +++++++++++++++++++++
chrome/test/BUILD.gn | 1 +
chrome/utility/BUILD.gn | 3 +-
chrome/utility/DEPS | 2 +-
chrome/utility/chrome_content_utility_client.cc | 21 +-
content/DEPS | 1 +
content/network/BUILD.gn | 6 +
content/network/DEPS | 1 -
.../network}/proxy_resolver_factory_mojo.cc | 37 +--
.../network}/proxy_resolver_factory_mojo.h | 14 +-
.../proxy_resolver_factory_mojo_unittest.cc | 75 ++++---
.../cpp => content/network}/proxy_service_mojo.cc | 9 +-
.../cpp => content/network}/proxy_service_mojo.h | 14 +-
content/public/network/BUILD.gn | 1 -
content/public/network/DEPS | 1 -
.../public/network}/mojo_proxy_resolver_factory.h | 14 +-
.../network/url_request_context_builder_mojo.cc | 4 +-
.../network/url_request_context_builder_mojo.h | 11 +-
content/public/test/DEPS | 1 -
content/test/BUILD.gn | 11 +-
content/test/DEPS | 1 +
.../test}/proxy_service_mojo_unittest.cc | 14 +-
.../test}/test_mojo_proxy_resolver_factory.cc | 17 +-
.../test}/test_mojo_proxy_resolver_factory.h | 19 +-
.../url_request_context_builder_mojo_unittest.cc | 4 +-
services/proxy_resolver/BUILD.gn | 46 ++--
services/proxy_resolver/OWNERS | 3 +
services/proxy_resolver/manifest.json | 14 ++
...tory_impl.cc => proxy_resolver_factory_impl.cc} | 61 ++---
...actory_impl.h => proxy_resolver_factory_impl.h} | 33 ++-
....cc => proxy_resolver_factory_impl_unittest.cc} | 43 ++--
...oxy_resolver_impl.cc => proxy_resolver_impl.cc} | 37 ++-
...proxy_resolver_impl.h => proxy_resolver_impl.h} | 14 +-
...unittest.cc => proxy_resolver_impl_unittest.cc} | 18 +-
services/proxy_resolver/proxy_resolver_service.cc | 51 +++++
services/proxy_resolver/proxy_resolver_service.h | 41 ++++
services/proxy_resolver/public/cpp/BUILD.gn | 40 ----
.../public/interfaces/proxy_resolver.mojom | 2 +
46 files changed, 732 insertions(+), 354 deletions(-)

Upstream: git.chromium.org


  • Share