admin/webrtc_m130
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

DefaultVideoQualityAnalyzer cleanup.

Browse Source 
Remove done todos

Bug: webrtc:10138
Change-Id: I33ad6da41bf51a0ed3bafd5ae671f94d00e16ce3
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/127563
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Artem Titov <titovartem@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#27109}
This commit is contained in:
Artem Titov 2019-03-13 14:05:05 +01:00 committed by Commit Bot
parent 982dc791db
commit baf271f978
1 changed files with 0 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
test/pc/e2e/analyzer/video/default_video_quality_analyzer.cc
View File

@ -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());