From 276cdfcf6aee3828e43f5123d2ea02ad8d024a20 Mon Sep 17 00:00:00 2001 From: Artem Titov Date: Tue, 19 Mar 2019 15:47:28 +0100 Subject: [PATCH] Rename resolution_of_encoded_image into resolution_of_rendered_frame. Rename resolution_of_encoded_image into resolution_of_rendered_frame in DefaultVideoQualityAnalyzer to make it consistent with the way, how it is calculated. Bug: webrtc:10138 Change-Id: Ibf89f08ac0646b57b4a6b8316cec1ed73bad02a2 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/128576 Reviewed-by: Mirko Bonadei Commit-Queue: Artem Titov Cr-Commit-Position: refs/heads/master@{#27179} --- test/pc/e2e/analyzer/video/default_video_quality_analyzer.cc | 4 ++-- test/pc/e2e/analyzer/video/default_video_quality_analyzer.h | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/test/pc/e2e/analyzer/video/default_video_quality_analyzer.cc b/test/pc/e2e/analyzer/video/default_video_quality_analyzer.cc index 239ec2c270..5f88a97a2f 100644 --- a/test/pc/e2e/analyzer/video/default_video_quality_analyzer.cc +++ b/test/pc/e2e/analyzer/video/default_video_quality_analyzer.cc @@ -433,7 +433,7 @@ void DefaultVideoQualityAnalyzer::ProcessComparison( } // Next stats can be calculated only if frame was received on remote side. if (!comparison.dropped) { - stats->resolution_of_encoded_image.AddSample( + stats->resolution_of_rendered_frame.AddSample( *comparison.frame_stats.rendered_frame_width * *comparison.frame_stats.rendered_frame_height); stats->transport_time_ms.AddSample( @@ -494,7 +494,7 @@ void DefaultVideoQualityAnalyzer::ReportResults(std::string test_case_name, ReportResult("time_between_freezes", test_case_name, stats.time_between_freezes_ms, "ms"); ReportResult("pixels_per_frame", test_case_name, - stats.resolution_of_encoded_image, "unitless"); + stats.resolution_of_rendered_frame, "unitless"); test::PrintResult("min_psnr", "", test_case_name, stats.psnr.IsEmpty() ? 0 : stats.psnr.GetMin(), "dB", /*important=*/false); diff --git a/test/pc/e2e/analyzer/video/default_video_quality_analyzer.h b/test/pc/e2e/analyzer/video/default_video_quality_analyzer.h index c51747f1ce..50295ac4c9 100644 --- a/test/pc/e2e/analyzer/video/default_video_quality_analyzer.h +++ b/test/pc/e2e/analyzer/video/default_video_quality_analyzer.h @@ -93,7 +93,7 @@ struct StreamStats { SamplesStatsCounter freeze_time_ms; // Mean time between one freeze end and next freeze start. SamplesStatsCounter time_between_freezes_ms; - SamplesStatsCounter resolution_of_encoded_image; + SamplesStatsCounter resolution_of_rendered_frame; int64_t dropped_by_encoder = 0; int64_t dropped_before_encoder = 0;