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

Side by Side 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
(Empty)
1 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file.
4
5 {
6 'conditions': [
7 ['OS=="android"', {
8 'targets': [
9 {
10 'target_name': 'native_test_apk',
11 'message': 'building native test package',
12 'type': 'none',
13 'actions': [
14 {
15 'action_name': 'native_test_apk',
16 'inputs': [
17 '<(DEPTH)/base/test/android/native_test_apk.xml',
18 '<!@(find test/android/java -name "*.java")',
19 'native_test_launcher.cc'
klobag.chromium 2012/03/28 00:28:57 Should we put native and Java in separate target?
20 ],
21 'outputs': [
22 # Awkwardly, we build a Debug APK even when gyp is in
23 # Release mode. I don't think it matters (e.g. we're
24 # probably happy to not codesign) but naming should be
25 # fixed.
26 '<(PRODUCT_DIR)/ChromeNativeTests-debug.apk',
klobag.chromium 2012/03/28 00:28:57 Why we have to call "-debug"?
27 ],
28 'action': [
29 'ant',
30 '-DPRODUCT_DIR=<(PRODUCT_DIR)',
31 '-buildfile',
32 '<(DEPTH)/base/test/android/native_test_apk.xml',
33 ]
34 }
35 ],
36 },
37 ],
38 }]
39 ]
40 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698