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

Unified Diff: content/content.gyp

Issue 23717023: Android: Add chrome-specific dynamic linker. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Trivial rebase. Created 7 years, 2 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: content/content.gyp
diff --git a/content/content.gyp b/content/content.gyp
index 221a1a53928a3e57c7edc508631022819b66a135..88d10b061ca211b11f6bce2d78f444fdfd731abf 100644
--- a/content/content.gyp
+++ b/content/content.gyp
@@ -494,6 +494,28 @@
},
'includes': [ 'content_jni.gypi' ],
},
+ {
+ 'target_name': 'content_android_linker',
+ 'type': 'shared_library',
+ 'conditions': [
+ ['android_webview_build == 0', {
+ # Avoid breaking the webview build because it doesn't have
+ # <(android_ndk_root)/crazy_linker.gyp. Note that it never uses
+ # the linker anyway.
+ 'sources': [
+ 'common/android/linker/linker_jni.cc',
+ ],
+ 'dependencies': [
+ '<(android_ndk_root)/crazy_linker.gyp:crazy_linker',
+ ],
+ 'include_dirs': [
+ # This is necessary to include base/android/sys_utils_constants.h
jochen (gone - plz use gerrit) 2013/10/09 12:08:23 shouldn't you depend on the target that has this h
digit1 2013/10/09 13:33:49 Actually no, it's a very special case: the header
jochen (gone - plz use gerrit) 2013/10/10 11:39:29 I'd either put this into a gyp variable, or introd
digit1 2013/10/10 12:21:41 Ok, I have added build/android/low_memory_device.g
+ '..',
+ ],
+ }],
+ ],
+ },
+
],
}], # OS == "android"
],

Powered by Google App Engine
This is Rietveld 408576698