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

Side by Side Diff: components/cronet/android/BUILD.gn

Issue 1839803002: Remove net & url small, iOS ICU alternatives, unit tests. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixed GN on iOS 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 2015 The Chromium Authors. All rights reserved. 1 # Copyright 2015 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 import("//build/buildflag_header.gni") 5 import("//build/buildflag_header.gni")
6 import("//build/config/android/config.gni") 6 import("//build/config/android/config.gni")
7 import("//build/config/android/rules.gni") 7 import("//build/config/android/rules.gni")
8 import("//build/util/version.gni") 8 import("//build/util/version.gni")
9 import("//chrome/version.gni") 9 import("//chrome/version.gni")
10 import("//testing/test.gni") 10 import("//testing/test.gni")
(...skipping 168 matching lines...) Expand 10 before | Expand all | Expand 10 after
179 if (defined(invoker.defines)) { 179 if (defined(invoker.defines)) {
180 defines += invoker.defines 180 defines += invoker.defines
181 } 181 }
182 182
183 if (defined(invoker.deps)) { 183 if (defined(invoker.deps)) {
184 deps += invoker.deps 184 deps += invoker.deps
185 } 185 }
186 } 186 }
187 } 187 }
188 188
189 # cronet_static_small target has reduced binary size through using
190 # ICU alternatives which requires file and ftp support be disabled.
191 cronet_static_tmpl("cronet_static_small") {
192 defines = [
193 "USE_ICU_ALTERNATIVES_ON_ANDROID=1",
194 "DISABLE_FILE_SUPPORT=1",
195 "DISABLE_FTP_SUPPORT=1",
196 ]
197
198 deps = [
199 "//net:net_small",
200 "//url:url_lib_use_icu_alternatives_on_android",
201 ]
202
203 if (cronet_enable_data_reduction_proxy_support) {
204 deps += [
205 "//components/data_reduction_proxy/core/browser:browser_small",
206 "//components/data_reduction_proxy/core/common:common_small",
207 ]
208 }
209 }
210
211 # cronet_static target depends on ICU and includes file and ftp support. 189 # cronet_static target depends on ICU and includes file and ftp support.
212 cronet_static_tmpl("cronet_static") { 190 cronet_static_tmpl("cronet_static") {
213 deps = [ 191 deps = [
214 "//base:i18n", 192 "//base:i18n",
215 "//net", 193 "//net",
216 "//url", 194 "//url",
217 ] 195 ]
218 196
219 if (cronet_enable_data_reduction_proxy_support) { 197 if (cronet_enable_data_reduction_proxy_support) {
220 deps += [ 198 deps += [
221 "//components/data_reduction_proxy/core/browser", 199 "//components/data_reduction_proxy/core/browser",
222 "//components/data_reduction_proxy/core/common", 200 "//components/data_reduction_proxy/core/common",
223 ] 201 ]
224 } 202 }
225 } 203 }
226 204
227 shared_library("cronet") { 205 shared_library("cronet") {
228 sources = [ 206 sources = [
229 "cronet_jni.cc", 207 "cronet_jni.cc",
230 ] 208 ]
231 deps = [ 209 deps = [
232 ":cronet_static_small", 210 ":cronet_static",
233 "//base", 211 "//base",
234 "//net:net_small", 212 "//net:net",
235 ] 213 ]
236 ldflags = [ "-Wl,--version-script=" + 214 ldflags = [ "-Wl,--version-script=" +
237 rebase_path("//components/cronet/android/only_jni_exports.lst") ] 215 rebase_path("//components/cronet/android/only_jni_exports.lst") ]
238 216
239 # Avoid hide_native_jni_exports as it adds another version script, and the 217 # Avoid hide_native_jni_exports as it adds another version script, and the
240 # ARM64 linker throws an error for multiple version scripts with anonymous 218 # ARM64 linker throws an error for multiple version scripts with anonymous
241 # version tags. 219 # version tags.
242 configs -= [ "//build/config/android:hide_native_jni_exports" ] 220 configs -= [ "//build/config/android:hide_native_jni_exports" ]
243 } 221 }
244 222
(...skipping 265 matching lines...) Expand 10 before | Expand all | Expand 10 after
510 testonly = true 488 testonly = true
511 489
512 # Need to use renaming_sources to maintain subdirectories. 490 # Need to use renaming_sources to maintain subdirectories.
513 renaming_sources = [ 491 renaming_sources = [
514 "test/assets/test/cacheable.txt", 492 "test/assets/test/cacheable.txt",
515 "test/assets/test/cacheable.txt.mock-http-headers", 493 "test/assets/test/cacheable.txt.mock-http-headers",
516 "test/assets/test/content_length_mismatch.html", 494 "test/assets/test/content_length_mismatch.html",
517 "test/assets/test/content_length_mismatch.html.mock-http-headers", 495 "test/assets/test/content_length_mismatch.html.mock-http-headers",
518 "test/assets/test/datareductionproxysuccess.txt", 496 "test/assets/test/datareductionproxysuccess.txt",
519 "test/assets/test/datareductionproxysuccess.txt.mock-http-headers", 497 "test/assets/test/datareductionproxysuccess.txt.mock-http-headers",
498 "test/assets/test/gzipped.html",
499 "test/assets/test/gzipped.html.mock-http-headers",
xunjieli 2016/03/31 13:59:40 Can we add the missing assets in a separate CL?
kapishnikov 2016/03/31 16:25:52 Done. https://codereview.chromium.org/1850443005/
520 "test/assets/test/multiredirect.html", 500 "test/assets/test/multiredirect.html",
521 "test/assets/test/multiredirect.html.mock-http-headers", 501 "test/assets/test/multiredirect.html.mock-http-headers",
522 "test/assets/test/notfound.html", 502 "test/assets/test/notfound.html",
523 "test/assets/test/notfound.html.mock-http-headers", 503 "test/assets/test/notfound.html.mock-http-headers",
524 "test/assets/test/quic_data/simple.txt", 504 "test/assets/test/quic_data/simple.txt",
525 "test/assets/test/redirect.html", 505 "test/assets/test/redirect.html",
526 "test/assets/test/redirect.html.mock-http-headers", 506 "test/assets/test/redirect.html.mock-http-headers",
527 "test/assets/test/redirect_invalid_scheme.html", 507 "test/assets/test/redirect_invalid_scheme.html",
528 "test/assets/test/redirect_invalid_scheme.html.mock-http-headers", 508 "test/assets/test/redirect_invalid_scheme.html.mock-http-headers",
529 "test/assets/test/sdch/dict/LeQxM80O", 509 "test/assets/test/sdch/dict/LeQxM80O",
(...skipping 347 matching lines...) Expand 10 before | Expand all | Expand 10 after
877 ":cronet_package_copy_native_lib", 857 ":cronet_package_copy_native_lib",
878 ":cronet_package_copy_native_lib_unstripped", 858 ":cronet_package_copy_native_lib_unstripped",
879 ":generate_javadoc", 859 ":generate_javadoc",
880 ":generate_licenses", 860 ":generate_licenses",
881 ":jar_cronet_api_source", 861 ":jar_cronet_api_source",
882 ":jar_cronet_other_source", 862 ":jar_cronet_other_source",
883 ":jar_cronet_sample_source", 863 ":jar_cronet_sample_source",
884 ":repackage_extracted_jars", 864 ":repackage_extracted_jars",
885 ] 865 ]
886 } 866 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698