apk-based test runner work. Not enabled yet. This CL is a combination of upstreaming, ndk/ant-ifi…

base/test/android/native_test.gyp

+# 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"', {

[Mark Mentovai, 2012/03/23 15:00:16]

Because a condition has a [ and { on the same line, I normally just close them
on the same line too, like
    ]},

That’ll probably clear up the stuff I bring up below about indentation.

[John Grabowski, 2012/03/24 01:59:18]

Done (2 cases)
Hopefully this works for you

+      'targets': [
+        {
+          'target_name': 'native_test_apk',
+          'message': 'building native test runner',

[bulach, 2012/03/23 10:30:48]

nit: when I think about "runner", the host side python infrastructure comes to
mind.. maybe "Building native test package" ?

[John Grabowski, 2012/03/24 01:59:18]

Done.

+          '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'
+              ],
+              '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',
+              ],
+              'action': [
+                'ant',
+                '-DPRODUCT_DIR=<(PRODUCT_DIR)',
+                '-buildfile',
+                '<(DEPTH)/base/test/android/native_test_apk.xml',
+              ]
+            },
+          ],
+        },
+      ],
+    }

[Mark Mentovai, 2012/03/23 15:00:16]

Something’s weird with the indentation here

+   ]
+  ]

[Mark Mentovai, 2012/03/23 15:00:16]

and here

+}
+

[Mark Mentovai, 2012/03/23 15:00:16]

Blank line at EOF.