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

Revert "Add framesRendered to StatsReport"

Browse Source 
This reverts commit dcfa938f9e768d463d3e336f4d014027504267dd.

Reason for revert: This CL blocks rolling WebRTC into chromium

Original change's description:
> Add framesRendered to StatsReport
> 
> 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}

TBR=steveanton@webrtc.org,solenberg@webrtc.org,joachimr@fb.com

Change-Id: Ia58feefd0ab557bb39ff79840dc8fa5004fee753
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: webrtc:9568
Reviewed-on: https://webrtc-review.googlesource.com/90900
Reviewed-by: Artem Titov <titovartem@webrtc.org>
Commit-Queue: Artem Titov <titovartem@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24134}
This commit is contained in:
Artem Titov
2018-07-27 14:52:57 +00:00
committed by Commit Bot
parent f81ce94176
commit 8a3ab0e7ea
4 changed files with 2 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
api/statstypes.cc
View File

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

1
api/statstypes.h
View File

@ -107,7 +107,6 @@ class StatsReport {
kStatsValueNameConcealmentEvents,
kStatsValueNameDataChannelId,
kStatsValueNameFramesDecoded,
kStatsValueNameFramesRendered,
kStatsValueNameFramesEncoded,
kStatsValueNameJitterBufferDelay,
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::kStatsValueNameTargetDelayMs, info.target_delay_ms},
{StatsReport::kStatsValueNameFramesDecoded, info.frames_decoded},
{StatsReport::kStatsValueNameFramesRendered, info.frames_rendered}};
};
for (const auto& i : ints)
report->AddInt(i.name, i.value);

6
pc/statscollector_unittest.cc
View File

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