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

Change reporting of time_between_freezes.

Browse Source 
Report time_between_freezes as test duration when there were no freezes
in the call.

Bug: webrtc:10138
Change-Id: I3d99be4b714f1b1d13e7b8b7055b368a20859490
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/141665
Reviewed-by: Ilya Nikolaevskiy <ilnik@webrtc.org>
Commit-Queue: Artem Titov <titovartem@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#28248}
This commit is contained in:
Artem Titov 2019-06-12 13:17:01 +02:00 committed by Commit Bot
parent 5cf390386b
commit da1c65fb53
2 changed files with 17 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

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

@ -81,7 +81,10 @@ void DefaultVideoQualityAnalyzer::Start(std::string test_case_name,
}
{
rtc::CritScope crit(&lock_);
RTC_CHECK(start_time_.IsMinusInfinity());
state_ = State::kActive;
start_time_ = Now();
}
}
@ -90,6 +93,13 @@ uint16_t DefaultVideoQualityAnalyzer::OnFrameCaptured(
const webrtc::VideoFrame& frame) {
// |next_frame_id| is atomic, so we needn't lock here.
uint16_t frame_id = next_frame_id_++;
Timestamp start_time = Timestamp::MinusInfinity();
{
rtc::CritScope crit(&lock_);
// Create a local copy of start_time_ to access it under |comparison_lock_|
// without holding a |lock_|
start_time = start_time_;
}
{
// Ensure stats for this stream exists.
rtc::CritScope crit(&comparison_lock_);
@ -98,7 +108,7 @@ uint16_t DefaultVideoQualityAnalyzer::OnFrameCaptured(
// Assume that the first freeze was before first stream frame captured.
// This way time before the first freeze would be counted as time between
// freezes.
stream_last_freeze_end_time_.insert({stream_label, Now()});
stream_last_freeze_end_time_.insert({stream_label, start_time});
}
}
{
@ -298,10 +308,12 @@ void DefaultVideoQualityAnalyzer::Stop() {
rtc::CritScope crit1(&lock_);
rtc::CritScope crit2(&comparison_lock_);
for (auto& item : stream_stats_) {
if (item.second.freeze_time_ms.IsEmpty()) {
continue;
}
const StreamState& state = stream_states_[item.first];
// If there are no freezes in the call we have to report
// time_between_freezes_ms as call duration and in such case
// |stream_last_freeze_end_time_| for this stream will be |start_time_|.
// If there is freeze, then we need add time from last rendered frame
// to last freeze end as time between freezes.
if (state.last_rendered_frame_time) {
item.second.time_between_freezes_ms.AddSample(
(state.last_rendered_frame_time.value() -

1
test/pc/e2e/analyzer/video/default_video_quality_analyzer.h
View File

@ -263,6 +263,7 @@ class DefaultVideoQualityAnalyzer : public VideoQualityAnalyzerInterface {
rtc::CriticalSection lock_;
State state_ RTC_GUARDED_BY(lock_) = State::kNew;
Timestamp start_time_ RTC_GUARDED_BY(lock_) = Timestamp::MinusInfinity();
// Frames that were captured by all streams and still aren't rendered by any
// stream or deemed dropped.
std::map<uint16_t, VideoFrame> captured_frames_in_flight_