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

Unified Diff: net/BUILD.gn

Issue 253603004: Make it so net/ can be built without file:// support. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Add TODO, minor cleanups Created 6 years, 8 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/BUILD.gn
===================================================================
--- net/BUILD.gn (revision 266235)
+++ net/BUILD.gn (working copy)
@@ -37,6 +37,8 @@
use_v8_in_net = false # TODO(brettw)!is_ios
enable_built_in_dns = !is_ios
+enable_file_support = 1
mef 2014/05/06 15:21:30 is there file support on ios?
mmenke 2014/05/06 18:13:06 I'm not sure, but the file file:// code all appear
+
enable_ftp_support = !is_ios
config("net_config") {
@@ -108,6 +110,19 @@
}
}
+ if (!enable_file_support) {
+ sources -= [
+ 'base/directory_lister.cc',
+ 'base/directory_lister.h',
+ 'url_request/url_request_file_dir_job.cc',
+ 'url_request/url_request_file_dir_job.h',
+ 'url_request/url_request_file_job.cc',
+ 'url_request/url_request_file_job.h',
+ 'url_request/file_protocol_handler.cc',
+ 'url_request/file_protocol_handler.h',
+ ]
+ }
+
if (!enable_ftp_support) {
sources -= [
"ftp/ftp_auth_cache.cc",
« no previous file with comments | « build/common.gypi ('k') | net/net.gyp » ('j') | net/url_request/url_request_unittest.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698