Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(567)

Unified Diff: net/proxy/proxy_service_mojo.cc

Issue 910343003: Create an in-process v8 proxy resolver using Mojo. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@v8-pac-complete-patch
Patch Set: Address review comments. Created 5 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: net/proxy/proxy_service_mojo.cc
diff --git a/net/proxy/proxy_service_mojo.cc b/net/proxy/proxy_service_mojo.cc
new file mode 100644
index 0000000000000000000000000000000000000000..8a25b2e9efac9de1ce245fbbaaeb9c555223c6f4
--- /dev/null
+++ b/net/proxy/proxy_service_mojo.cc
@@ -0,0 +1,59 @@
+// Copyright 2015 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.
+
+#include "net/proxy/proxy_service_mojo.h"
+
+#include "base/logging.h"
+#include "net/dns/host_resolver_mojo.h"
+#include "net/interfaces/proxy_resolver_service.mojom.h"
+#include "net/proxy/in_process_mojo_proxy_resolver_factory.h"
+#include "net/proxy/mojo_proxy_resolver_impl.h"
+#include "net/proxy/proxy_resolver_mojo.h"
+#include "net/proxy/proxy_resolver_v8_tracing.h"
+#include "net/proxy/proxy_service.h"
+
+namespace net {
+
+// static
Sam McNally 2015/03/02 03:26:24 These aren't really static.
Anand Mistry (off Chromium) 2015/03/03 02:54:43 Removed.
+ProxyService* CreateProxyServiceUsingMojoFactory(
+ MojoProxyResolverFactory* mojo_proxy_factory,
+ ProxyConfigService* proxy_config_service,
+ ProxyScriptFetcher* proxy_script_fetcher,
+ DhcpProxyScriptFetcher* dhcp_proxy_script_fetcher,
+ HostResolver* host_resolver,
+ NetLog* net_log,
+ NetworkDelegate* network_delegate) {
+ DCHECK(proxy_config_service);
+ DCHECK(proxy_script_fetcher);
+ DCHECK(dhcp_proxy_script_fetcher);
+ DCHECK(host_resolver);
+
+ ProxyResolverMojo* proxy_resolver =
+ new ProxyResolverMojo(mojo_proxy_factory, host_resolver);
+
+ ProxyService* proxy_service =
+ new ProxyService(proxy_config_service, proxy_resolver, net_log);
+
+ // Configure fetchers to use for PAC script downloads and auto-detect.
+ proxy_service->SetProxyScriptFetchers(proxy_script_fetcher,
+ dhcp_proxy_script_fetcher);
+
+ return proxy_service;
+}
+
+// static
+ProxyService* CreateProxyServiceUsingMojoInProcess(
+ ProxyConfigService* proxy_config_service,
+ ProxyScriptFetcher* proxy_script_fetcher,
+ DhcpProxyScriptFetcher* dhcp_proxy_script_fetcher,
+ HostResolver* host_resolver,
+ NetLog* net_log,
+ NetworkDelegate* network_delegate) {
+ return CreateProxyServiceUsingMojoFactory(
+ InProcessMojoProxyResolverFactory::GetInstance(), proxy_config_service,
+ proxy_script_fetcher, dhcp_proxy_script_fetcher, host_resolver, net_log,
+ network_delegate);
+}
+
+} // namespace net

Powered by Google App Engine
This is Rietveld 408576698