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

Unified Diff: base/test/android/native_test.gyp

Issue 9834037: apk-based test runner work. Not enabled yet. This CL is a combination of upstreaming, ndk/ant-ifi… (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: one more line tweak Created 8 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
Index: base/test/android/native_test.gyp
diff --git a/base/test/android/native_test.gyp b/base/test/android/native_test.gyp
new file mode 100644
index 0000000000000000000000000000000000000000..86d372859fef9d6a4cc8111730fc37a2367a9b69
--- /dev/null
+++ b/base/test/android/native_test.gyp
@@ -0,0 +1,40 @@
+# Copyright (c) 2012 The Chromium Authors. All rights reserved.
+# Use of this source code is governed by a BSD-style license that can be
+# found in the LICENSE file.
+
+{
+ 'conditions': [
+ ['OS=="android"', {
+ 'targets': [
+ {
+ 'target_name': 'native_test_apk',
+ 'message': 'building native test package',
+ 'type': 'none',
+ 'actions': [
+ {
+ 'action_name': 'native_test_apk',
+ 'inputs': [
+ '<(DEPTH)/base/test/android/native_test_apk.xml',
+ '<!@(find test/android/java -name "*.java")',
+ 'native_test_launcher.cc'
klobag.chromium 2012/03/28 00:28:57 Should we put native and Java in separate target?
+ ],
+ 'outputs': [
+ # Awkwardly, we build a Debug APK even when gyp is in
+ # Release mode. I don't think it matters (e.g. we're
+ # probably happy to not codesign) but naming should be
+ # fixed.
+ '<(PRODUCT_DIR)/ChromeNativeTests-debug.apk',
klobag.chromium 2012/03/28 00:28:57 Why we have to call "-debug"?
+ ],
+ 'action': [
+ 'ant',
+ '-DPRODUCT_DIR=<(PRODUCT_DIR)',
+ '-buildfile',
+ '<(DEPTH)/base/test/android/native_test_apk.xml',
+ ]
+ }
+ ],
+ },
+ ],
+ }]
+ ]
+}

Powered by Google App Engine
This is Rietveld 408576698