tree 2e3015e5644e51412e46cafce546acc7fc300301
parent af317971af59b9c6b96ded939798b8985f4aa4da
author Jeremy Leconte <jleconte@google.com> 1753173580 -0700
committer WebRTC LUCI CQ <webrtc-scoped@luci-project-accounts.iam.gserviceaccount.com> 1753173785 -0700

Revert "Allow StdoutMetricsExporter to merge stream and peers info in test_name."

This reverts commit 772386c50321aabfcaeea75857515d6f939c6ac1.

Reason for revert: breaks chromium.fyi ios bots

Bug: None
Original change's description:
> Allow StdoutMetricsExporter to merge stream and peers info in test_name.
>
> Currently, metrics printed by StdoutMetricsExporter have lost info
> about stream, sender/receiver and peers. They look like:
>
> RESULT psnr_dB: foreman_cif_net_delay_0_0_plr_0_VP9= {38.697086,1.7471016} unitless_biggerIsBetter
>
> Which makes it impossible to know the stream to which this PSNR
> measurement was referring to.
>
> This CL tries to restore the behaviour before [1], which changed test
> names with:
>
> std::string DefaultVideoQualityAnalyzer::GetTestCaseName(
>     const std::string& stream_label) const {
>   if (!absl::GetFlag(FLAGS_isolated_script_test_perf_output).empty()) {
>     return test_label_ + "/" + stream_label;
>   }
>   return test_label_;
> }
>
> [1] - https://webrtc-review.googlesource.com/c/src/+/383720
>
> Bug: None
> Change-Id: Iad3cf5cb723e37c2a32fc510a2d484b796024ace
> Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/399280
> Reviewed-by: Jeremy Leconte <jleconte@google.com>
> Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
> Cr-Commit-Position: refs/heads/main@{#45185}

Bug: None
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Change-Id: I09690c949d4b395cadf5b201d37094f0310fadc9
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/401500
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Commit-Queue: Jeremy Leconte <jleconte@google.com>
Cr-Commit-Position: refs/heads/main@{#45189}