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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: content/content_shell.gypi

Issue 147533004: Remove unneeded JNI registrations. (Closed) Base URL: https://git.chromium.org/chromium/src.git@master
Patch Set: Fix android webview build issues. Created 6 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
« chrome/chrome_android.gypi ('K') | « content/content_browser.gypi ('k') | content/shell/android/browsertests_apk/content_browser_tests_android.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/content_shell.gypi
diff --git a/content/content_shell.gypi b/content/content_shell.gypi
index 5879e2d58967f8ffad04ee529a92c129e3928a64..b78ba10796f09909ef1520db079e44ac92d8b5d5 100644
--- a/content/content_shell.gypi
+++ b/content/content_shell.gypi
@@ -997,6 +997,19 @@
'-lgabi++', # For rtti
],
}],
+ ['component == "static_library"', {
+ 'ldflags!': [
+ # Don't need to strip symbols from libs, because exports are controlled
+ # through linker vesrion script.
+ '-Wl,--exclude-libs=ALL',
+ ],
+ 'ldflags': [
+ # Android exports in linker version script.
+ # TODO(ostap): Gyp doesn't corectly calculate path if <(DEPTH) is used here.
+ # Use <(DEPTH) when gyp is fixed or replaced with gn.
+ '-Wl,--version-script="../../build/android/android_exports.lst"',
+ ],
+ }],
],
},
{
« chrome/chrome_android.gypi ('K') | « content/content_browser.gypi ('k') | content/shell/android/browsertests_apk/content_browser_tests_android.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698