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

Unified Diff: libyuv_test.gyp

Issue 1398383003: Add libyuv_unittest_apk target to run tests on Android (Closed) Base URL: https://chromium.googlesource.com/libyuv/libyuv@master
Patch Set: Created 5 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
« no previous file with comments | « DEPS ('k') | setup_links.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: libyuv_test.gyp
diff --git a/libyuv_test.gyp b/libyuv_test.gyp
index fad36debd897abfcd6c8095ff2a7295299b47462..fd4a98b256576c3c797d33fab57b6818664bf933 100644
--- a/libyuv_test.gyp
+++ b/libyuv_test.gyp
@@ -14,12 +14,10 @@
'targets': [
{
'target_name': 'libyuv_unittest',
- 'type': 'executable',
+ 'type': '<(gtest_target_type)',
'dependencies': [
'libyuv.gyp:libyuv',
- # The tests are based on gtest
'testing/gtest.gyp:gtest',
- 'testing/gtest.gyp:gtest_main',
],
'defines': [
# Enable the following 3 macros to turn off assembly for specified CPU.
@@ -77,6 +75,11 @@
'HAVE_JPEG',
],
}],
+ ['OS=="android"', {
+ 'dependencies': [
+ '<(DEPTH)/testing/android/native_test.gyp:native_test_native_code',
+ ],
+ }],
# TODO(YangZhang): These lines can be removed when high accuracy
# YUV to RGB to Neon is ported.
[ '(target_arch == "armv7" or target_arch == "armv7s" \
@@ -166,6 +169,40 @@
],
},
], # targets
+ 'conditions': [
+ ['OS=="android"', {
+ 'targets': [
+ {
+ # TODO(kjellander): Figure out what to change in build/apk_test.gypi
+ # to it can be used instead of the copied code below. Using it in its
+ # current version was not possible, since the target starts with 'lib',
+ # which somewhere confuses the variables.
+ 'target_name': 'libyuv_unittest_apk',
+ 'type': 'none',
+ 'variables': {
+ # These are used to configure java_apk.gypi included below.
+ 'test_type': 'gtest',
+ 'apk_name': 'libyuv_unittest',
+ 'intermediate_dir': '<(PRODUCT_DIR)/libyuv_unittest_apk',
+ 'final_apk_path': '<(intermediate_dir)/libyuv_unittest-debug.apk',
+ 'java_in_dir': '<(DEPTH)/testing/android/native_test/java',
+ 'native_lib_target': 'libyuv_unittest',
+ 'gyp_managed_install': 0,
+ },
+ 'includes': [ 'build/java_apk.gypi' ],
+ 'dependencies': [
+ '<(DEPTH)/base/base.gyp:base_java',
+ '<(DEPTH)/build/android/pylib/device/commands/commands.gyp:chromium_commands',
+ '<(DEPTH)/build/android/pylib/remote/device/dummy/dummy.gyp:remote_device_dummy_apk',
+ '<(DEPTH)/testing/android/appurify_support.gyp:appurify_support_java',
+ '<(DEPTH)/testing/android/on_device_instrumentation.gyp:reporter_java',
+ '<(DEPTH)/tools/android/android_tools.gyp:android_tools',
+ 'libyuv_unittest',
+ ],
+ },
+ ],
+ }],
+ ],
}
# Local Variables:
« no previous file with comments | « DEPS ('k') | setup_links.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698