Exclude test types that don't upload from the flakiness dashboard UI.

Exclude test types that don't upload from the flakiness dashboard UI.

Moves some logic from the JS code in to the code that generates
builders.jsonp for steps with test in the name that don't actually
run tests. Also expands this list a bit.

Also, creates a new list for test steps that do run tests, but
don't upload to the dashboard. In a future patch, I'll expand this
list and give it a nice UI in the flakiness dashboard that points
people to a way to get the test step to upload results.

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=175829

[Dirk Pranke, 2014-06-09 19:06:12]

lgtm

https://codereview.chromium.org/309783002/diff/1/Tools/TestResultServer/handl...
File Tools/TestResultServer/handlers/buildershandler.py (right):

https://codereview.chromium.org/309783002/diff/1/Tools/TestResultServer/handl...
Tools/TestResultServer/handlers/buildershandler.py:175: for name in
NON_TEST_STEP_NAMES:
to replace lines 174-183:

if not 'test' in step_name:
  continue

if any(n in step_name for n in NON_TEST_STEP_NAMES):
  continue

if re.search(...):
  continue

?

[ojan, 2014-06-09 21:39:17]

https://codereview.chromium.org/309783002/diff/1/Tools/TestResultServer/handl...
File Tools/TestResultServer/handlers/buildershandler.py (right):

https://codereview.chromium.org/309783002/diff/1/Tools/TestResultServer/handl...
Tools/TestResultServer/handlers/buildershandler.py:175: for name in
NON_TEST_STEP_NAMES:
Indeed! Also, I realized I forgot to run the unittests. Updated the unittest to
make sure to hit each continue in the new code.

https://codereview.chromium.org/309783002/diff/20001/Tools/TestResultServer/h...
File Tools/TestResultServer/handlers/buildershandler_unittest.py (left):

https://codereview.chromium.org/309783002/diff/20001/Tools/TestResultServer/h...
Tools/TestResultServer/handlers/buildershandler_unittest.py:1: #!/usr/bin/env
python
Removed this to make presubmit happy. Presumably I could also have fixed this by
setting the executable bit.

[commit-bot: I haz the power, 2014-06-09 21:40:16]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/ojan@chromium.org/309783002/20001

[commit-bot: I haz the power, 2014-06-09 22:43:59]

Change committed as 175829