WebRTC Android APK trybot recipe

scripts/slave/recipes/webrtc_android_apk_trybot.py

Index: scripts/slave/recipes/webrtc_android_apk_trybot.py
diff --git a/scripts/slave/recipes/webrtc_android_apk_trybot.py b/scripts/slave/recipes/webrtc_android_apk_trybot.py
new file mode 100644
index 0000000000000000000000000000000000000000..fa680acc8dd95ad232885c75bbc84332e9e2070f
--- /dev/null
+++ b/scripts/slave/recipes/webrtc_android_apk_trybot.py
@@ -0,0 +1,59 @@
+# Copyright 2013 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.
+
+DEPS = [
+  'chromium',
+  'gclient',
+  'json',
+  'path',
+  'properties',
+  'python',
+  'step',
+  'webrtc',
+]
+
+
+def GenSteps(api):
+  config_vals = {}
+  config_vals.update(
+    dict((str(k),v) for k,v in api.properties.iteritems() if k.isupper())
+  )
+  api.webrtc.set_config('webrtc_android_apk_try_builder', **config_vals)
+  api.step.auto_resolve_conflicts = True
+
+  yield api.gclient.checkout()
+  yield api.webrtc.apply_svn_patch()
+  yield api.webrtc.envsetup()
+  yield api.webrtc.runhooks()
+  yield api.chromium.cleanup_temp()
+  yield api.webrtc.compile()
+
+  for test in api.webrtc.get_normal_tests():
+    # The libjingle tests are not yet supported on Android.
+    if not test.startswith('libjingle'):
+      yield api.webrtc.runtests(test)
+
+
+def GenTests(api):
+  for build_config in ('Debug', 'Release'):
+    props = api.properties(
+      HOST_PLATFORM='linux',
+      HOST_ARCH='intel',
+      HOST_BITS=64,

[iannucci, 2013/11/18 18:49:26]

In general, you should let the HOST_* be set automatically. You can 'simulate'
the host platform by adding e.g. api.platform('linux', 64) to your test (you'll
need to add platform to DEPS)

[kjellander_chromium, 2013/11/20 14:29:16]

Ah, so that's how it works. I remember seeing the platform recipe module when I
started looking a recipes and I thought "great, finally moving out all that into
a single place". Then I forgot about it on the way... I'll be happy to use it
for what it's meant for.

+      TARGET_PLATFORM='linux',

[iannucci, 2013/11/18 18:49:26]

target platform should actually be 'android', right?

[kjellander_chromium, 2013/11/20 14:29:16]

Yes.

+      TARGET_ARCH='arm',
+      TARGET_BITS=64,
+      BUILD_CONFIG=build_config,
+      ANDROID_APK=True,
+      revision='12345',
+      patch_url='try_job_svn_patch'
+    )
+
+    yield (
+      api.test(build_config) +
+      props +
+      api.step_data('envsetup',
+                    api.json.output({'PATH': './',})
+      )
+    )