zqz/platform-external-webrtc
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Add framesRendered to StatsReport

Browse Source 
Bug: webrtc:9568
Change-Id: I6976f4c48b67f6a81f57260a91966debbef38eb4
Reviewed-on: https://webrtc-review.googlesource.com/90840
Reviewed-by: Steve Anton <steveanton@webrtc.org>
Commit-Queue: Steve Anton <steveanton@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24121}
This commit is contained in:
Joachim Reiersen
2018-07-26 12:58:20 -07:00
committed by Commit Bot
parent 01560dead9
commit dcfa938f9e
4 changed files with 9 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
api/statstypes.cc
View File

@ -400,6 +400,8 @@ const char* StatsReport::Value::display_name() const {
return "datachannelid"; return "datachannelid";
case kStatsValueNameFramesDecoded: case kStatsValueNameFramesDecoded:
return "framesDecoded"; return "framesDecoded";
case kStatsValueNameFramesRendered:
return "framesRendered";
case kStatsValueNameFramesEncoded: case kStatsValueNameFramesEncoded:
return "framesEncoded"; return "framesEncoded";
case kStatsValueNameJitterBufferDelay: case kStatsValueNameJitterBufferDelay:

1
api/statstypes.h
View File

@ -107,6 +107,7 @@ class StatsReport {
kStatsValueNameConcealmentEvents, kStatsValueNameConcealmentEvents,
kStatsValueNameDataChannelId, kStatsValueNameDataChannelId,
kStatsValueNameFramesDecoded, kStatsValueNameFramesDecoded,
kStatsValueNameFramesRendered,
kStatsValueNameFramesEncoded, kStatsValueNameFramesEncoded,
kStatsValueNameJitterBufferDelay, kStatsValueNameJitterBufferDelay,
kStatsValueNameMediaType, kStatsValueNameMediaType,

2
pc/statscollector.cc
View File

@ -275,7 +275,7 @@ void ExtractStats(const cricket::VideoReceiverInfo& info, StatsReport* report) {
{StatsReport::kStatsValueNameRenderDelayMs, info.render_delay_ms}, {StatsReport::kStatsValueNameRenderDelayMs, info.render_delay_ms},
{StatsReport::kStatsValueNameTargetDelayMs, info.target_delay_ms}, {StatsReport::kStatsValueNameTargetDelayMs, info.target_delay_ms},
{StatsReport::kStatsValueNameFramesDecoded, info.frames_decoded}, {StatsReport::kStatsValueNameFramesDecoded, info.frames_decoded},
}; {StatsReport::kStatsValueNameFramesRendered, info.frames_rendered}};
for (const auto& i : ints) for (const auto& i : ints)
report->AddInt(i.name, i.value); report->AddInt(i.name, i.value);

6
pc/statscollector_unittest.cc
View File

@ -1741,7 +1741,8 @@ TEST_P(StatsCollectorTrackTest, VerifyVideoReceiveSsrcStatsNew) {
VideoReceiverInfo video_receiver_info; VideoReceiverInfo video_receiver_info;
video_receiver_info.add_ssrc(1234); video_receiver_info.add_ssrc(1234);
video_receiver_info.frames_decoded = 10; video_receiver_info.frames_decoded = 10;
video_receiver_info.qp_sum = 11; video_receiver_info.frames_rendered = 11;
video_receiver_info.qp_sum = 12;
VideoMediaInfo video_info; VideoMediaInfo video_info;
video_info.receivers.push_back(video_receiver_info); video_info.receivers.push_back(video_receiver_info);
@ -1755,6 +1756,9 @@ TEST_P(StatsCollectorTrackTest, VerifyVideoReceiveSsrcStatsNew) {
EXPECT_EQ(rtc::ToString(video_receiver_info.frames_decoded), EXPECT_EQ(rtc::ToString(video_receiver_info.frames_decoded),
ExtractSsrcStatsValue(reports, ExtractSsrcStatsValue(reports,
StatsReport::kStatsValueNameFramesDecoded)); StatsReport::kStatsValueNameFramesDecoded));
EXPECT_EQ(rtc::ToString(video_receiver_info.frames_rendered),
ExtractSsrcStatsValue(reports,
StatsReport::kStatsValueNameFramesRendered));
EXPECT_EQ(rtc::ToString(*video_receiver_info.qp_sum), EXPECT_EQ(rtc::ToString(*video_receiver_info.qp_sum),
ExtractSsrcStatsValue(reports, StatsReport::kStatsValueNameQpSum)); ExtractSsrcStatsValue(reports, StatsReport::kStatsValueNameQpSum));
} }