From baf271f978303004185c2a774c2cd63bb38a4696 Mon Sep 17 00:00:00 2001 From: Artem Titov Date: Wed, 13 Mar 2019 14:05:05 +0100 Subject: [PATCH] DefaultVideoQualityAnalyzer cleanup. Remove done todos Bug: webrtc:10138 Change-Id: I33ad6da41bf51a0ed3bafd5ae671f94d00e16ce3 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/127563 Reviewed-by: Mirko Bonadei Commit-Queue: Artem Titov Cr-Commit-Position: refs/heads/master@{#27109} --- test/pc/e2e/analyzer/video/default_video_quality_analyzer.cc | 2 -- 1 file changed, 2 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 601cd2bc78..239ec2c270 100644 --- a/test/pc/e2e/analyzer/video/default_video_quality_analyzer.cc +++ b/test/pc/e2e/analyzer/video/default_video_quality_analyzer.cc @@ -135,7 +135,6 @@ void DefaultVideoQualityAnalyzer::OnFrameEncoded( uint16_t frame_id, const webrtc::EncodedImage& encoded_image) { rtc::CritScope crit(&lock_); - // TODO(titovartem) we need to pick right spatial index here. auto it = frame_stats_.find(frame_id); RTC_DCHECK(it != frame_stats_.end()); RTC_DCHECK(it->second.encoded_time.IsInfinite()) @@ -154,7 +153,6 @@ void DefaultVideoQualityAnalyzer::OnFrameDropped( void DefaultVideoQualityAnalyzer::OnFrameReceived( uint16_t frame_id, const webrtc::EncodedImage& input_image) { - // TODO(titovartem) We should always receive only single spatial layer here. rtc::CritScope crit(&lock_); auto it = frame_stats_.find(frame_id); RTC_DCHECK(it != frame_stats_.end());