Make step status EXCEPTION when test runner has infrastructure failure.

scripts/slave/recipe_modules/chromium_android/config.py

Index: scripts/slave/recipe_modules/chromium_android/config.py
diff --git a/scripts/slave/recipe_modules/chromium_android/config.py b/scripts/slave/recipe_modules/chromium_android/config.py
index d75d1a0a1aa3ffbffb1affde629287479fe5bd50..22f3f9b7aabea45ef972f7d0cef16a55a0d1d6a3 100644
--- a/scripts/slave/recipe_modules/chromium_android/config.py
+++ b/scripts/slave/recipe_modules/chromium_android/config.py
@@ -29,6 +29,7 @@ def BaseConfig(INTERNAL=False, REPO_NAME=None, REPO_URL=None,
     tests = List(inner_type=basestring),
     cr_build_android = Static(Path('[CHECKOUT]', 'build', 'android')),
     test_runner = Single(Path),
+    test_runner_exit_codes = Dict(value_type=int),

[luqui, 2015/03/02 23:40:58]

This doesn't feel right to me.  I would prefer to hardcode this dictionary in
the test_runner function -- keeps the implementation details from leaking all
over the place.

Unless you expect different configurations to use different dictionaries, which
I find unlikely.

[mikecase (-- gone --), 2015/03/03 00:46:47]

Done.

     gclient_custom_deps = Dict(value_type=(basestring, types.NoneType)),
     storage_bucket = Single(basestring),
     channel = Single(basestring, empty_val='chrome'),
@@ -58,6 +59,7 @@ config_ctx = config_item_context(BaseConfig, VAR_TEST_MAP, TEST_NAME_FORMAT)
 def base_config(c):
   c.internal_dir_name = 'clank'
   c.test_runner = Path('[CHECKOUT]', 'build', 'android', 'test_runner.py')
+  c.test_runner_exit_codes = {'ERROR': 1, 'INFRA': 87, 'WARNING': 88}
 
 @config_ctx()
 def main_builder(c):