Add suite information to the JSON reporter.

Add suite information to the JSON reporter.

Closes #380

R=alexander.doroshko@gmail.com, kevmoo@google.com

Committed: https://github.com/dart-lang/test/commit/bdeff621359170be3ef58c60e1acfef8ef6dc423

[nweiz, 2016-01-27 01:04:51]

nweiz@google.com changed reviewers:
+ alexander.doroshko@jetbrains.com, kevmoo@google.com

[alexander.doroshko, 2016-01-27 10:19:29]

alexander.doroshko@gmail.com changed reviewers:
+ alexander.doroshko@gmail.com

[alexander.doroshko, 2016-01-27 10:19:30]

lgtm

[alexander.doroshko, 2016-01-27 10:19:32]

lgtm

lgtm

[kevmoo, 2016-01-28 20:40:49]

lgtm

[nweiz, 2016-01-28 21:41:46]

Description was changed from

==========
Add suite information to the JSON reporter.

Closes #380
==========

to

==========
Add suite information to the JSON reporter.

Closes #380

R=alexander.doroshko@gmail.com, kevmoo@google.com

Committed:
https://github.com/dart-lang/test/commit/bdeff621359170be3ef58c60e1acfef8ef6d...
==========

[nweiz, 2016-01-28 21:41:48]

Committed patchset #3 (id:40001) manually as
bdeff621359170be3ef58c60e1acfef8ef6dc423 (presubmit successful).

[alexander.doroshko, 2016-02-11 14:26:43]

https://codereview.chromium.org/1637193003/diff/40001/json_reporter.md
File json_reporter.md (right):

https://codereview.chromium.org/1637193003/diff/40001/json_reporter.md#newcod...
json_reporter.md:309: doesn't exist at all). Its path is relative to the root of
the current package.
Spec says that Suite.path is relative to the current package root, but in fact
we see that absolute paths are emitted. 
Current implementation in IntelliJ IDEA works fine with absolute paths, so we'd
be happy with just a spec update.
If you believe that JSON reporter should be changed to emit relative paths
please let us know asap as IntelliJ IDEA and WebStorm releases are very soon.

[nweiz, 2016-02-12 00:58:25]

https://codereview.chromium.org/1637193003/diff/40001/json_reporter.md
File json_reporter.md (right):

https://codereview.chromium.org/1637193003/diff/40001/json_reporter.md#newcod...
json_reporter.md:309: doesn't exist at all). Its path is relative to the root of
the current package.
On 2016/02/11 14:26:43, alexander.doroshko wrote:
> Spec says that Suite.path is relative to the current package root, but in fact
> we see that absolute paths are emitted. 
> Current implementation in IntelliJ IDEA works fine with absolute paths, so
we'd
> be happy with just a spec update.
> If you believe that JSON reporter should be changed to emit relative paths
> please let us know asap as IntelliJ IDEA and WebStorm releases are very soon.

Good to know. I put out https://codereview.chromium.org/1690053004 to update the
docs.

I suspect what happens here is that we emit absolute paths if we're passed
absolute paths and vice versa, but I don't want to guarantee that behavior.