GTTF: Include info about disabled tests and platforms in JSON test summary.

GTTF: Include info about disabled tests and platforms in JSON test summary.

This will make it possible to analyze which tests are disabled on specific
platforms, and whether the failures are really platform-specific.

Including list of all tests in the summary makes it possible to e.g. detect
when a test first appeared (or disappeared), which platforms a test is compiled
on, or whether some tests have been filtered out.

BUG=236893
R=sky@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=243232

[sky, 2014-01-06 19:24:15]

LGTM

https://codereview.chromium.org/93603008/diff/1/base/test/launcher/test_launc...
File base/test/launcher/test_launcher.cc (right):

https://codereview.chromium.org/93603008/diff/1/base/test/launcher/test_launc...
base/test/launcher/test_launcher.cc:688: #ifdef NDEBUG
nit: #if defined(NDEBUG)

https://codereview.chromium.org/93603008/diff/1/base/test/launcher/test_resul...
File base/test/launcher/test_results_tracker.cc (right):

https://codereview.chromium.org/93603008/diff/1/base/test/launcher/test_resul...
base/test/launcher/test_results_tracker.cc:172:
disabled_tests_.insert(test_name_no_disabled);
nit: refactor 170-171 into function above. I'm thinking this and 164 would
become:

disabled_tests_.insert(RemovedDisabled(test_name));

[Paweł Hajdan Jr., 2014-01-07 09:12:27]

https://codereview.chromium.org/93603008/diff/1/base/test/launcher/test_launc...
File base/test/launcher/test_launcher.cc (right):

https://codereview.chromium.org/93603008/diff/1/base/test/launcher/test_launc...
base/test/launcher/test_launcher.cc:688: #ifdef NDEBUG
On 2014/01/06 19:24:15, sky wrote:
> nit: #if defined(NDEBUG)

Done.

https://codereview.chromium.org/93603008/diff/1/base/test/launcher/test_resul...
File base/test/launcher/test_results_tracker.cc (right):

https://codereview.chromium.org/93603008/diff/1/base/test/launcher/test_resul...
base/test/launcher/test_results_tracker.cc:172:
disabled_tests_.insert(test_name_no_disabled);
On 2014/01/06 19:24:15, sky wrote:
> nit: refactor 170-171 into function above. I'm thinking this and 164 would
> become:
> 
> disabled_tests_.insert(RemovedDisabled(test_name));

Done.

[Paweł Hajdan Jr., 2014-01-07 09:26:29]

Committed patchset #2 manually as r243232.