[tools] Include full benchmark measurement name in error reporting

[tools] Include full benchmark measurement name in error reporting

For benchmarks with multiple measurements (e.g. runtime, compile
time, generated code size), in case of error, we'd get only the
last portion of the name ("Compile"), which is ambiguous and makes
debugging dificult. This change includes the full name, e.g.
"Unity/Physics/Compile".

BUG=

Review-Url: https://codereview.chromium.org/2615623003
Cr-Commit-Position: refs/heads/master@{#42078}
Committed: https://chromium.googlesource.com/v8/v8/+/9e5a064197ca3db0ff39bcc04767494ead4aa929

[Mircea Trofin, 2017-01-04 17:34:03]

Description was changed from

==========
[tools] Include full benchmark measurement name in error reporting

For benchmarks with multiple measurements (e.g. runtime, compile
time, generated code size), in case of error, we'd get only the
last portion of the name ("Compile"), which is ambiguous and makes
debugging dificult. This change includes the full name, e.g.
"Unity/Physics/Compile".

BUG=
==========

to

==========
[tools] Include full benchmark measurement name in error reporting

For benchmarks with multiple measurements (e.g. runtime, compile
time, generated code size), in case of error, we'd get only the
last portion of the name ("Compile"), which is ambiguous and makes
debugging dificult. This change includes the full name, e.g.
"Unity/Physics/Compile".

BUG=
==========

[Mircea Trofin, 2017-01-04 17:34:04]

mtrofin@chromium.org changed reviewers:
+ bradnelson@chromium.org, machenbach@chromium.org

[bradnelson, 2017-01-04 23:15:46]

The CQ bit was checked by bradnelson@chromium.org

[bradnelson, 2017-01-04 23:15:46]

lgtm

[commit-bot: I haz the power, 2017-01-04 23:15:52]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-04 23:36:06]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1483571746087700, "parent_rev":
"731af7a48801788bb66f080bbebac3c6b8c7cc11", "commit_rev":
"9e5a064197ca3db0ff39bcc04767494ead4aa929"}

[commit-bot: I haz the power, 2017-01-04 23:36:11]

Description was changed from

==========
[tools] Include full benchmark measurement name in error reporting

For benchmarks with multiple measurements (e.g. runtime, compile
time, generated code size), in case of error, we'd get only the
last portion of the name ("Compile"), which is ambiguous and makes
debugging dificult. This change includes the full name, e.g.
"Unity/Physics/Compile".

BUG=
==========

to

==========
[tools] Include full benchmark measurement name in error reporting

For benchmarks with multiple measurements (e.g. runtime, compile
time, generated code size), in case of error, we'd get only the
last portion of the name ("Compile"), which is ambiguous and makes
debugging dificult. This change includes the full name, e.g.
"Unity/Physics/Compile".

BUG=

Review-Url: https://codereview.chromium.org/2615623003
Cr-Commit-Position: refs/heads/master@{#42078}
Committed:
https://chromium.googlesource.com/v8/v8/+/9e5a064197ca3db0ff39bcc04767494ead4...
==========

[commit-bot: I haz the power, 2017-01-04 23:36:11]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/v8/v8/+/9e5a064197ca3db0ff39bcc04767494ead4...

[Michael Achenbach, 2017-01-05 15:01:59]

lgtm