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

Side by Side Diff: net/net.gyp

Issue 1839803002: Remove net & url small, iOS ICU alternatives, unit tests. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Addressed Misha's comments. Created 4 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 unified diff | Download patch
OLDNEW
1 # Copyright 2013 The Chromium Authors. All rights reserved. 1 # Copyright 2013 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 { 5 {
6 'variables': { 6 'variables': {
7 'chromium_code': 1, 7 'chromium_code': 1,
8 'linux_link_kerberos%': 0, 8 'linux_link_kerberos%': 0,
9 'conditions': [ 9 'conditions': [
10 ['chromeos==1 or embedded==1 or OS=="ios"', { 10 ['chromeos==1 or embedded==1 or OS=="ios"', {
(...skipping 88 matching lines...) Expand 10 before | Expand all | Expand 10 after
99 '../build/protoc.gypi', 99 '../build/protoc.gypi',
100 ], 100 ],
101 'defines': [ 101 'defines': [
102 'NET_IMPLEMENTATION', 102 'NET_IMPLEMENTATION',
103 ], 103 ],
104 }, 104 },
105 { 105 {
106 # GN version: //net 106 # GN version: //net
107 'target_name': 'net', 107 'target_name': 'net',
108 'dependencies': [ 108 'dependencies': [
109 '../base/base.gyp:base_i18n',
110 '../third_party/brotli/brotli.gyp:brotli',
111 '../third_party/icu/icu.gyp:icui18n',
112 '../third_party/icu/icu.gyp:icuuc',
113 '../third_party/protobuf/protobuf.gyp:protobuf_lite',
114 '../url/url.gyp:url_lib', 109 '../url/url.gyp:url_lib',
115 'net_quic_proto',
116 ],
117 'sources': [
118 'base/filename_util_icu.cc',
119 'base/net_string_util_icu.cc',
120 'filter/brotli_filter.cc',
121 ], 110 ],
122 'includes': [ 'net_common.gypi' ], 111 'includes': [ 'net_common.gypi' ],
112
113 'conditions': [
114 # ICU Alternatives
115 ['use_platform_icu_alternatives == 1', {
116 'defines': [
117 'USE_PLATFORM_ICU_ALTERNATIVES=1',
118 ],
119 'conditions': [
120 ['OS == "android"', {
121 'sources': [
122 'base/net_string_util_icu_alternatives_android.cc',
123 'base/net_string_util_icu_alternatives_android.h',
124 ],
125 }],
126 ['OS == "ios"', {
127 'sources': [
128 'base/net_string_util_icu_alternatives_ios.mm',
129 ],
130 }],
131 ],
132 },
133 # 'use_platform_icu_alternatives != 1'
134 {
135 'sources': [
136 'base/filename_util_icu.cc',
137 'base/net_string_util_icu.cc',
138 ],
139 'dependencies': [
140 '../base/base.gyp:base_i18n',
141 '../third_party/icu/icu.gyp:icui18n',
142 '../third_party/icu/icu.gyp:icuuc',
143 '../third_party/protobuf/protobuf.gyp:protobuf_lite',
144 'net_quic_proto',
145 ],
146 }],
147 # Brotli support.
148 ['disable_brotli_filter == 1', {
149 'sources': [
150 'filter/brotli_filter_disabled.cc',
151 ],
152 },
153 # 'disable_brotli_filter != 1'
154 {
155 'sources': [
156 'filter/brotli_filter.cc',
157 ],
158 'dependencies': [
159 '../third_party/brotli/brotli.gyp:brotli',
160 ],
161 }],
162 ],
123 }, 163 },
124 { 164 {
125 # GN version: //net:net_unittests 165 # GN version: //net:net_unittests
126 'target_name': 'net_unittests', 166 'target_name': 'net_unittests',
127 'type': '<(gtest_target_type)', 167 'type': '<(gtest_target_type)',
128 'dependencies': [ 168 'dependencies': [
129 '../base/base.gyp:base', 169 '../base/base.gyp:base',
130 '../base/base.gyp:base_i18n', 170 '../base/base.gyp:base_i18n',
131 '../base/third_party/dynamic_annotations/dynamic_annotations.gyp:dynamic _annotations', 171 '../base/third_party/dynamic_annotations/dynamic_annotations.gyp:dynamic _annotations',
132 '../crypto/crypto.gyp:crypto', 172 '../crypto/crypto.gyp:crypto',
(...skipping 80 matching lines...) Expand 10 before | Expand all | Expand 10 after
213 }], 253 }],
214 [ 'use_kerberos==0 or OS == "android"', { 254 [ 'use_kerberos==0 or OS == "android"', {
215 # These are excluded on Android, because the actual Kerberos support, 255 # These are excluded on Android, because the actual Kerberos support,
216 # which these test, is in a separate app on Android. 256 # which these test, is in a separate app on Android.
217 'sources!': [ 257 'sources!': [
218 'http/http_auth_gssapi_posix_unittest.cc', 258 'http/http_auth_gssapi_posix_unittest.cc',
219 'http/mock_gssapi_library_posix.cc', 259 'http/mock_gssapi_library_posix.cc',
220 'http/mock_gssapi_library_posix.h', 260 'http/mock_gssapi_library_posix.h',
221 ], 261 ],
222 }], 262 }],
223 [ 'use_kerberos==0', { 263 [ 'use_kerberos==0', {
224 'sources!': [ 264 'sources!': [
225 'http/http_auth_handler_negotiate_unittest.cc', 265 'http/http_auth_handler_negotiate_unittest.cc',
226 ], 266 ],
227 }], 267 }],
228 [ 'use_nss_verifier == 0', { 268 [ 'use_nss_verifier == 0', {
229 # Only include this test when using NSS for cert verification. 269 # Only include this test when using NSS for cert verification.
230 'sources!': [ 270 'sources!': [
231 'cert_net/nss_ocsp_unittest.cc', 271 'cert_net/nss_ocsp_unittest.cc',
232 ], 272 ],
233 }], 273 }],
(...skipping 178 matching lines...) Expand 10 before | Expand all | Expand 10 after
412 # icu. Figure out a way to remove that dependency. 452 # icu. Figure out a way to remove that dependency.
413 'dependencies': [ 453 'dependencies': [
414 '../testing/android/native_test.gyp:native_test_native_code', 454 '../testing/android/native_test.gyp:native_test_native_code',
415 ] 455 ]
416 }], 456 }],
417 ['use_v8_in_net==1 and v8_use_external_startup_data==1', { 457 ['use_v8_in_net==1 and v8_use_external_startup_data==1', {
418 'dependencies': [ 458 'dependencies': [
419 '../gin/gin.gyp:gin', 459 '../gin/gin.gyp:gin',
420 ] 460 ]
421 }], 461 }],
462 # Unit tests that are not supported by the current ICU alternatives on A ndroid.
463 ['OS == "android" and use_platform_icu_alternatives == 1', {
464 'sources!': [
465 'base/filename_util_unittest.cc',
466 'url_request/url_request_job_unittest.cc',
467 ],
468 }],
469 # Unit tests that are not supported by the current ICU alternatives on i OS.
470 ['OS == "ios" and use_platform_icu_alternatives == 1', {
471 'sources!': [
472 'base/filename_util_unittest.cc',
473 'base/url_util_unittest.cc',
474 'cert/x509_certificate_unittest.cc',
475 'socket/ssl_client_socket_pool_unittest.cc',
476 'http/http_auth_handler_basic_unittest.cc',
477 'http/http_auth_handler_digest_unittest.cc',
478 'http/http_auth_handler_factory_unittest.cc',
479 'http/http_auth_unittest.cc',
480 'http/http_content_disposition_unittest.cc',
481 'http/http_network_transaction_unittest.cc',
482 'http/http_proxy_client_socket_pool_unittest.cc',
483 'spdy/spdy_network_transaction_unittest.cc',
484 'spdy/spdy_proxy_client_socket_unittest.cc',
485 'url_request/url_request_job_unittest.cc',
486 'url_request/url_request_unittest.cc',
487 ],
488 }],
489 # Exclude brotli test if the support for brotli is disabled.
490 ['disable_brotli_filter == 1', {
491 'sources!': [
492 'filter/brotli_filter_unittest.cc',
493 ],
494 }],
422 ], 495 ],
423 'target_conditions': [ 496 'target_conditions': [
424 # These source files are excluded by default platform rules, but they 497 # These source files are excluded by default platform rules, but they
425 # are needed in specific cases on other platforms. Re-including them can 498 # are needed in specific cases on other platforms. Re-including them can
426 # only be done in target_conditions as it is evaluated after the 499 # only be done in target_conditions as it is evaluated after the
427 # platform rules. 500 # platform rules.
428 ['OS == "android"', { 501 ['OS == "android"', {
429 'sources/': [ 502 'sources/': [
430 ['include', '^base/address_tracker_linux_unittest\\.cc$'], 503 ['include', '^base/address_tracker_linux_unittest\\.cc$'],
431 ], 504 ],
(...skipping 895 matching lines...) Expand 10 before | Expand all | Expand 10 after
1327 'simple_quic_tools', 1400 'simple_quic_tools',
1328 ], 1401 ],
1329 'sources': [ 1402 'sources': [
1330 'tools/quic/quic_server_bin.cc', 1403 'tools/quic/quic_server_bin.cc',
1331 ], 1404 ],
1332 }, 1405 },
1333 ] 1406 ]
1334 }], 1407 }],
1335 ['OS=="android"', { 1408 ['OS=="android"', {
1336 'targets': [ 1409 'targets': [
1337 { # The same target as 'net', but with smaller binary size due to
1338 # exclusion of ICU, FTP, FILE and WebSockets support.
1339 'target_name': 'net_small',
1340 'variables': {
1341 'disable_ftp_support': 1,
1342 'disable_file_support': 1,
1343 'enable_websockets': 0,
1344 },
1345 'dependencies': [
1346 '../url/url.gyp:url_lib_use_icu_alternatives_on_android',
1347 ],
1348 'defines': [
1349 'USE_ICU_ALTERNATIVES_ON_ANDROID=1',
1350 'DISABLE_FILE_SUPPORT=1',
1351 'DISABLE_FTP_SUPPORT=1',
1352 ],
1353 'sources': [
1354 'filter/brotli_filter_disabled.cc',
1355 'base/net_string_util_icu_alternatives_android.cc',
1356 'base/net_string_util_icu_alternatives_android.h',
1357 ],
1358 'includes': [ 'net_common.gypi' ],
1359 },
1360 { 1410 {
1361 'target_name': 'net_jni_headers', 1411 'target_name': 'net_jni_headers',
1362 'type': 'none', 1412 'type': 'none',
1363 'sources': [ 1413 'sources': [
1364 'android/java/src/org/chromium/net/AndroidCertVerifyResult.java', 1414 'android/java/src/org/chromium/net/AndroidCertVerifyResult.java',
1365 'android/java/src/org/chromium/net/AndroidKeyStore.java', 1415 'android/java/src/org/chromium/net/AndroidKeyStore.java',
1366 'android/java/src/org/chromium/net/AndroidNetworkLibrary.java', 1416 'android/java/src/org/chromium/net/AndroidNetworkLibrary.java',
1367 'android/java/src/org/chromium/net/AndroidTrafficStats.java', 1417 'android/java/src/org/chromium/net/AndroidTrafficStats.java',
1368 'android/java/src/org/chromium/net/GURLUtils.java', 1418 'android/java/src/org/chromium/net/GURLUtils.java',
1369 'android/java/src/org/chromium/net/HttpNegotiateAuthenticator.java', 1419 'android/java/src/org/chromium/net/HttpNegotiateAuthenticator.java',
(...skipping 356 matching lines...) Expand 10 before | Expand all | Expand 10 after
1726 '../build/isolate.gypi', 1776 '../build/isolate.gypi',
1727 ], 1777 ],
1728 'sources': [ 1778 'sources': [
1729 'net_unittests.isolate', 1779 'net_unittests.isolate',
1730 ], 1780 ],
1731 }, 1781 },
1732 ], 1782 ],
1733 }], 1783 }],
1734 ], 1784 ],
1735 } 1785 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698