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

Unified Diff: net/net.gyp

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: Flag. 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/net.gyp
diff --git a/net/net.gyp b/net/net.gyp
index 0237da848650b3390d02f0d188047477af4cab48..7539bfe07cbd1be9de6c667df12859ef3ff2127a 100644
--- a/net/net.gyp
+++ b/net/net.gyp
@@ -720,6 +720,7 @@
'dns/mojo_host_resolver_impl_unittest.cc',
'proxy/mojo_proxy_resolver_factory_impl_unittest.cc',
'proxy/mojo_proxy_resolver_impl_unittest.cc',
+ 'proxy/mojo_proxy_service_unittest.cc',
Sam McNally 2015/02/26 06:36:40 proxy_service_mojo_unittest.cc
Anand Mistry (off Chromium) 2015/02/27 03:00:58 Done.
],
},
],
@@ -1270,9 +1271,13 @@
'sources': [
'dns/mojo_host_resolver_impl.cc',
'dns/mojo_host_resolver_impl.h',
+ 'proxy/in_process_mojo_proxy_resolver_factory.cc',
+ 'proxy/in_process_mojo_proxy_resolver_factory.h',
'proxy/mojo_proxy_resolver_factory.h',
'proxy/proxy_resolver_mojo.cc',
'proxy/proxy_resolver_mojo.h',
+ 'proxy/proxy_service_mojo.cc',
+ 'proxy/proxy_service_mojo.h',
],
'dependencies': [
'mojo_type_converters',
@@ -1280,6 +1285,13 @@
'net_interfaces',
'../mojo/mojo_base.gyp:mojo_environment_chromium',
'../third_party/mojo/mojo_public.gyp:mojo_cpp_bindings',
+
+ # NOTE(amistry): As long as we support in-process Mojo v8 PAC, we
+ # need this dependency since in_process_mojo_proxy_resolver_factory
+ # creates the utility process side Mojo services in the browser
+ # process. Ultimately, this will go away when we only support
+ # out-of-process.
+ 'net_utility_services',
],
},
{

Powered by Google App Engine
This is Rietveld 408576698