Remove global MetricsLoggerAndExporter instance in favor of MetricsLogger
Bug: b/246095034
Change-Id: Ie3dd5947f0f593bd17cfecfa333d5254fa40769d
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/276628
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Artem Titov <titovartem@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#38190}
diff --git a/api/test/metrics/global_metrics_logger_and_exporter.cc b/api/test/metrics/global_metrics_logger_and_exporter.cc
index 476dd4d..915dcac 100644
--- a/api/test/metrics/global_metrics_logger_and_exporter.cc
+++ b/api/test/metrics/global_metrics_logger_and_exporter.cc
@@ -21,11 +21,6 @@
namespace webrtc {
namespace test {
-namespace {
-
-MetricsLoggerAndExporter* global_metrics_logger_and_exporter = nullptr;
-
-} // namespace
MetricsLogger* GetGlobalMetricsLogger() {
static MetricsLogger* logger_ =
@@ -44,21 +39,5 @@
return success;
}
-MetricsLoggerAndExporter* GetGlobalMetricsLoggerAndExporter() {
- return global_metrics_logger_and_exporter;
-}
-
-void SetupGlobalMetricsLoggerAndExporter(
- std::vector<std::unique_ptr<MetricsExporter>> exporters) {
- RTC_CHECK(global_metrics_logger_and_exporter == nullptr);
- global_metrics_logger_and_exporter = new MetricsLoggerAndExporter(
- Clock::GetRealTimeClock(), std::move(exporters));
-}
-
-void ExportAndDestroyGlobalMetricsLoggerAndExporter() {
- RTC_CHECK(global_metrics_logger_and_exporter != nullptr);
- delete global_metrics_logger_and_exporter;
-}
-
} // namespace test
} // namespace webrtc
diff --git a/api/test/metrics/global_metrics_logger_and_exporter.h b/api/test/metrics/global_metrics_logger_and_exporter.h
index 9032eb1..3f9bcec 100644
--- a/api/test/metrics/global_metrics_logger_and_exporter.h
+++ b/api/test/metrics/global_metrics_logger_and_exporter.h
@@ -26,20 +26,6 @@
bool ExportPerfMetric(MetricsLogger& logger,
std::vector<std::unique_ptr<MetricsExporter>> exporters);
-// Returns current global `MetricsLoggerAndExporter`. Returns `nullptr` if there
-// is global instance wasn't initialized.
-MetricsLoggerAndExporter* GetGlobalMetricsLoggerAndExporter();
-
-// Initialize global `MetricsLoggerAndExporter` with provided vector of
-// exporters. Crashes if there is already initialized global instance.
-void SetupGlobalMetricsLoggerAndExporter(
- std::vector<std::unique_ptr<MetricsExporter>> exporters);
-
-// Destroys global `MetricsLoggerAndExporter` forcing it to export all collected
-// metrics to the configured exporters. Crashes if there is no initialized
-// global instance.
-void ExportAndDestroyGlobalMetricsLoggerAndExporter();
-
} // namespace test
} // namespace webrtc