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

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

Issue 2633593004: Android: Use linker script to hide all non-JNI symbols (Closed)
Patch Set: sub-repos now updated Created 3 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 unified diff | Download patch
« no previous file with comments | « chrome/android/BUILD.gn ('k') | components/cronet/android/only_jni_exports.lst » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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/process_version.gni") 8 import("//build/util/process_version.gni")
9 import("//build/util/version.gni") 9 import("//build/util/version.gni")
10 import("//testing/test.gni") 10 import("//testing/test.gni")
(...skipping 246 matching lines...) Expand 10 before | Expand all | Expand 10 after
257 257
258 shared_library("cronet") { 258 shared_library("cronet") {
259 sources = [ 259 sources = [
260 "cronet_jni.cc", 260 "cronet_jni.cc",
261 ] 261 ]
262 deps = [ 262 deps = [
263 ":cronet_static", 263 ":cronet_static",
264 "//base", 264 "//base",
265 "//net:net", 265 "//net:net",
266 ] 266 ]
267 ldflags = [ "-Wl,--version-script=" +
268 rebase_path("//components/cronet/android/only_jni_exports.lst") ]
269
270 # Avoid hide_native_jni_exports as it adds another version script, and the
271 # ARM64 linker throws an error for multiple version scripts with anonymous
272 # version tags.
273 configs -= [ "//build/config/android:hide_native_jni_exports" ]
274 } 267 }
275 268
276 cronet_api_srcjar_deps = [ ":cronet_api_version_srcjar" ] 269 cronet_api_srcjar_deps = [ ":cronet_api_version_srcjar" ]
277 270
278 # cronet_api_java.jar defines Cronet API. 271 # cronet_api_java.jar defines Cronet API.
279 android_library("cronet_api_java") { 272 android_library("cronet_api_java") {
280 output_name = "cronet_api" 273 output_name = "cronet_api"
281 java_files = [ 274 java_files = [
282 "api/src/org/chromium/net/BidirectionalStream.java", 275 "api/src/org/chromium/net/BidirectionalStream.java",
283 "api/src/org/chromium/net/CronetEngine.java", 276 "api/src/org/chromium/net/CronetEngine.java",
(...skipping 261 matching lines...) Expand 10 before | Expand all | Expand 10 after
545 "//base/test:test_support", 538 "//base/test:test_support",
546 "//components/prefs", 539 "//components/prefs",
547 "//net", 540 "//net",
548 "//net:simple_quic_tools", 541 "//net:simple_quic_tools",
549 "//net:test_support", 542 "//net:test_support",
550 "//third_party/icu", 543 "//third_party/icu",
551 ] 544 ]
552 545
553 include_dirs = [ _cronet_version_header_include_dir ] 546 include_dirs = [ _cronet_version_header_include_dir ]
554 547
555 ldflags = [ "-Wl,--version-script=" +
556 rebase_path("//components/cronet/android/only_jni_exports.lst") ]
557
558 # Avoid hide_native_jni_exports as it adds another version script, and the
559 # ARM64 linker throws an error for multiple version scripts with anonymous
560 # version tags.
561 configs -= [ "//build/config/android:hide_native_jni_exports" ]
562
563 if (cronet_enable_data_reduction_proxy_support) { 548 if (cronet_enable_data_reduction_proxy_support) {
564 deps += [ "//components/data_reduction_proxy/core/browser:browser_small" ] 549 deps += [ "//components/data_reduction_proxy/core/browser:browser_small" ]
565 } 550 }
566 } 551 }
567 552
568 android_resources("cronet_test_apk_resources") { 553 android_resources("cronet_test_apk_resources") {
569 testonly = true 554 testonly = true
570 resource_dirs = [ 555 resource_dirs = [
571 "test/res", 556 "test/res",
572 "test/smoketests/res/native", 557 "test/smoketests/res/native",
(...skipping 924 matching lines...) Expand 10 before | Expand all | Expand 10 after
1497 deps = [ 1482 deps = [
1498 ":cronet_package_copy_native_test_lib", 1483 ":cronet_package_copy_native_test_lib",
1499 ":cronet_package_copy_native_test_lib_unstripped", 1484 ":cronet_package_copy_native_test_lib_unstripped",
1500 ":cronet_package_copy_test_assets", 1485 ":cronet_package_copy_test_assets",
1501 ":cronet_package_copy_test_files", 1486 ":cronet_package_copy_test_files",
1502 ":cronet_package_copy_test_support_apks", 1487 ":cronet_package_copy_test_support_apks",
1503 ":repackage_extracted_test_jars", 1488 ":repackage_extracted_test_jars",
1504 ] 1489 ]
1505 } 1490 }
1506 } 1491 }
OLDNEW
« no previous file with comments | « chrome/android/BUILD.gn ('k') | components/cronet/android/only_jni_exports.lst » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698