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

Unified Diff: net/BUILD.gn

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: Rebase Created 5 years, 9 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
« no previous file with comments | « chrome/common/chrome_switches.cc ('k') | net/net.gyp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/BUILD.gn
diff --git a/net/BUILD.gn b/net/BUILD.gn
index e9e673767ee630a05d7e9bf9148b63e1c88644cd..0a126e32be5f57eaccabdf77a64c203b8a3e4270 100644
--- a/net/BUILD.gn
+++ b/net/BUILD.gn
@@ -798,9 +798,13 @@ if (use_v8_in_net && !is_android) {
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",
]
public_deps = [
@@ -810,6 +814,12 @@ if (use_v8_in_net && !is_android) {
"//mojo/common",
"//net/interfaces",
"//third_party/mojo/src/mojo/public/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",
]
}
@@ -950,6 +960,7 @@ if (!is_ios && !is_android) {
deps = [
":net", # TODO(brettw) bug 363749: this shouldn't be necessary. It's not
# in the GYP build, and can be removed when the bug is fixed.
+
eroman 2015/03/11 21:08:19 extra space added
Anand Mistry (off Chromium) 2015/03/12 04:51:42 Removed.
":test_support",
"//base",
"//base/test:test_support",
@@ -1411,6 +1422,7 @@ if (!is_android && !is_win && !is_mac) {
"proxy/mojo_proxy_resolver_factory_impl_unittest.cc",
"proxy/mojo_proxy_resolver_impl_unittest.cc",
"proxy/proxy_resolver_mojo_unittest.cc",
+ "proxy/proxy_service_mojo_unittest.cc",
]
}
« no previous file with comments | « chrome/common/chrome_switches.cc ('k') | net/net.gyp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698