commit e05bf45198d602ab409fcff7958fcd854d625247
author Artem Titov <titovartem@webrtc.org> Sat Sep 24 23:25:15 2022
committer WebRTC LUCI CQ <webrtc-scoped@luci-project-accounts.iam.gserviceaccount.com> Mon Sep 26 08:46:01 2022
tree 86d7c27cb7812d311531c58aff28acc2b9b36aad
parent dd65499002603d16d408193b88d4b8bd12e29b7b

Migrate video_send_stream_tests.cc to the new perf metrics export API

Bug: b/246095034
Change-Id: I185381205dd5233990307b872971a9397f126d2c
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/276631
Reviewed-by: Tomas Gunnarsson <tommi@webrtc.org>
Commit-Queue: Artem Titov <titovartem@webrtc.org>
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#38195}

video/BUILD.gn

diff --git a/video/BUILD.gn b/video/BUILD.gn
index 828803b..2c8e268 100644
--- a/video/BUILD.gn
+++ b/video/BUILD.gn
@@ -794,6 +794,8 @@
       "../api/rtc_event_log",
       "../api/task_queue",
       "../api/task_queue:default_task_queue_factory",
+      "../api/test/metrics:global_metrics_logger_and_exporter",
+      "../api/test/metrics:metric",
       "../api/test/video:function_video_factory",
       "../api/units:data_rate",
       "../api/units:frequency",
@@ -887,7 +889,6 @@
       "../test:mock_frame_transformer",
       "../test:mock_transport",
       "../test:null_transport",
-      "../test:perf_test",
       "../test:rtp_test_utils",
       "../test:run_loop",
       "../test:scoped_key_value_config",

video/video_send_stream_tests.cc

diff --git a/video/video_send_stream_tests.cc b/video/video_send_stream_tests.cc
index f4ba3df..965073b 100644
--- a/video/video_send_stream_tests.cc
+++ b/video/video_send_stream_tests.cc
@@ -16,6 +16,8 @@
 #include "api/sequence_checker.h"
 #include "api/task_queue/default_task_queue_factory.h"
 #include "api/task_queue/task_queue_base.h"
+#include "api/test/metrics/global_metrics_logger_and_exporter.h"
+#include "api/test/metrics/metric.h"
 #include "api/test/simulated_network.h"
 #include "api/video/builtin_video_bitrate_allocator_factory.h"
 #include "api/video/encoded_image.h"
@@ -66,7 +68,6 @@
 #include "test/gtest.h"
 #include "test/null_transport.h"
 #include "test/rtcp_packet_parser.h"
-#include "test/testsupport/perf_test.h"
 #include "test/video_encoder_proxy_factory.h"
 #include "video/send_statistics_proxy.h"
 #include "video/transport_adapter.h"
@@ -1514,9 +1515,11 @@
           EXPECT_EQ(1u, stats.substreams.size());
           int total_bitrate_bps =
               stats.substreams.begin()->second.total_bitrate_bps;
-          test::PrintResult(
-              "bitrate_stats_", "min_transmit_bitrate_low_remb", "bitrate_bps",
-              static_cast<size_t>(total_bitrate_bps), "bps", false);
+          test::GetGlobalMetricsLogger()->LogSingleValueMetric(
+              "bitrate_stats_min_transmit_bitrate_low_remb", "bitrate_bps",
+              static_cast<size_t>(total_bitrate_bps) / 1000.0,
+              test::Unit::kKilobitsPerSecond,
+              test::ImprovementDirection::kNeitherIsBetter);
           if (total_bitrate_bps > kHighBitrateBps) {
             rtp_rtcp_->SetRemb(kRembBitrateBps, {ssrc});
             bitrate_capped_ = true;