Refactor the way that TestApi works so that it is actually useful.

scripts/slave/recipe_modules/chromium_android/config.py

 # 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.
 
 from slave.recipe_configs_util import config_item_context, ConfigGroup
 from slave.recipe_configs_util import DictConfig, ListConfig, SimpleConfig
 from slave.recipe_configs_util import StaticConfig
 
 def BaseConfig(INTERNAL, REPO_NAME, REPO_URL, **_kwargs):
   return ConfigGroup(
     INTERNAL = StaticConfig(INTERNAL),
     REPO_NAME = StaticConfig(REPO_NAME),
     REPO_URL = StaticConfig(REPO_URL),
     target_arch = SimpleConfig(basestring, required=False, empty_val=''),
-    custom_vars = DictConfig(value_type=basestring),
     extra_env = DictConfig(value_type=(basestring,int,list)),
     run_findbugs = SimpleConfig(bool, required=False, empty_val=False),
     run_lint = SimpleConfig(bool, required=False, empty_val=False),
     run_checkdeps = SimpleConfig(bool, required=False, empty_val=False)
   )
 
 
 VAR_TEST_MAP = {
   'INTERNAL': [True, False],
   'REPO_NAME': ['src', 'internal'],
@@ skipping 34 matching lines @@
     'ANDROID_SDK_BUILD_TOOLS_VERSION': 'android-KeyLimePie',
     'ANDROID_SDK_ROOT': ['third_party', 'android_tools_internal', 'sdk'],
     'ANDROID_SDK_VERSION': 'KeyLimePie'
   }
 
 @config_ctx()
 def try_builder(c):
   if c.INTERNAL:
     c.run_findbugs = True
     c.run_lint = True