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

scripts/slave/recipes/blink_trybot.expected/warn_on_flakey.json

 [
   {
     "cmd": [
       "python",
       "-u",
       "[DEPOT_TOOLS_ROOT]/gclient.py",
       "config",
       "--spec",
       "cache_dir = None\nsolutions = [{'custom_deps': {'src/third_party/WebKit': 'svn://svn-mirror.golo.chromium.org/blink/trunk'}, 'custom_vars': {'googlecode_url': 'svn://svn-mirror.golo.chromium.org/%s', 'nacl_trunk': 'svn://svn-mirror.golo.chromium.org/native_client/trunk', 'sourceforge_url': 'svn://svn-mirror.golo.chromium.org/%(repo)s', 'webkit_revision': 'HEAD', 'webkit_trunk': 'svn://svn-mirror.golo.chromium.org/blink/trunk'}, 'deps_file': 'DEPS', 'managed': True, 'name': 'src', 'url': 'svn://svn-mirror.golo.chromium.org/chrome/trunk/src'}, {'custom_deps': {'src/chrome/test/data/firefox2_profile/searchplugins': None, 'src/chrome/test/data/firefox2_searchplugins': None, 'src/chrome/test/data/firefox3_profile/searchplugins': None, 'src/chrome/test/data/firefox3_searchplugins': None, 'src/chrome/test/data/osdd': None, 'src/chrome/test/data/perf/frame_rate/private': None, 'src/chrome/test/data/plugin': None, 'src/chrome/test/data/ssl/certs': None, 'src/data/autodiscovery': None, 'src/data/esctf': None, 'src/data/mach_ports': None, 'src/data/memory_test': None, 'src/data/mozilla_js_tests': None, 'src/data/page_cycler': None, 'src/data/selenium_core': None, 'src/data/tab_switching': None, 'src/tools/grit/grit/test/data': None}, 'deps_file': 'DEPS', 'managed': True, 'name': 'src-internal', 'url': 'svn://svn-mirror.golo.chromium.org/chrome-internal/trunk/src-internal'}]"
     ],
@@ skipping 184 matching lines @@
       "Release",
       "-o",
       "[SLAVE_BUILD_ROOT]/layout-test-results",
       "--build-dir",
       "[SLAVE_BUILD_ROOT]/src/out",
       "--json-test-results",
       "/path/to/tmp/json"
     ],
     "name": "webkit_tests (with patch)",
     "~followup_annotations": [
+      "step returned non-zero exit code: 1",
       "@@@STEP_TEXT@<br/>unexpected_flakes:<br/>flake/totally-flakey.html<br/><br/>unexpected_failures:<br/>bad/totally-bad-probably.html<br/>tricky/totally-maybe-not-awesome.html<br/><br/>Total executed: 9001@@@",
       "@@@STEP_LOG_LINE@json.test_results@{@@@",
       "@@@STEP_LOG_LINE@json.test_results@  \"num_passes\": 9001, @@@",
       "@@@STEP_LOG_LINE@json.test_results@  \"tests\": {@@@",
       "@@@STEP_LOG_LINE@json.test_results@    \"bad\": {@@@",
       "@@@STEP_LOG_LINE@json.test_results@      \"totally-bad-probably.html\": {@@@",
       "@@@STEP_LOG_LINE@json.test_results@        \"actual\": \"FAIL\", @@@",
       "@@@STEP_LOG_LINE@json.test_results@        \"expected\": \"PASS\", @@@",
       "@@@STEP_LOG_LINE@json.test_results@        \"is_unexpected\": true@@@",
       "@@@STEP_LOG_LINE@json.test_results@      }@@@",
@@ skipping 180 matching lines @@
       "@@@STEP_LOG_LINE@python.inline@if failures['ignored']:@@@",
       "@@@STEP_LOG_LINE@python.inline@  print 'Ignored failures:'@@@",
       "@@@STEP_LOG_LINE@python.inline@  print '\\n'.join(failures['ignored'])@@@",
       "@@@STEP_LOG_LINE@python.inline@@@@",
       "@@@STEP_LOG_LINE@python.inline@sys.exit(bool(failures['new']))@@@",
       "@@@STEP_LOG_END@python.inline@@@",
       "@@@STEP_FAILURE@@@"
     ]
   }
 ]