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

Remove deprecated VideoSendStream::StreamStats data members.

Browse Source 
Bug: webrtc:10198
Change-Id: Ie48727acc6d1c9af42f3a997c98d9fdab4675d4a
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/173622
Reviewed-by: Magnus Flodman <mflodman@webrtc.org>
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#31080}
This commit is contained in:
Mirko Bonadei 2020-04-15 20:57:37 +02:00 committed by Commit Bot
parent 3d687a1e3e
commit ee1e6bcb02
2 changed files with 0 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
call/video_send_stream.h
View File

@ -83,13 +83,6 @@ class VideoSendStream {
// A snapshot of the most recent Report Block with additional data of
// interest to statistics. Used to implement RTCRemoteInboundRtpStreamStats.
absl::optional<ReportBlockData> report_block_data;
// These booleans are redundant; this information is already exposed in
// |type|.
// TODO(hbos): Update downstream projects to use |type| instead and delete
// these members.
bool is_flexfec = false;
bool is_rtx = false;
};
struct Stats {

6
video/send_statistics_proxy.cc
View File

@ -792,18 +792,12 @@ VideoSendStream::StreamStats* SendStatisticsProxy::GetStatsEntry(
case VideoSendStream::StreamStats::StreamType::kRtx:
entry->referenced_media_ssrc =
rtp_config_.GetMediaSsrcAssociatedWithRtxSsrc(ssrc);
entry->is_rtx = true;
break;
case VideoSendStream::StreamStats::StreamType::kFlexfec:
entry->referenced_media_ssrc =
rtp_config_.GetMediaSsrcAssociatedWithFlexfecSsrc(ssrc);
entry->is_flexfec = true;
break;
}
// TODO(hbos): Remove these booleans once downstream projects stop depedning
// on them, reading the value of |type| instead.
RTC_DCHECK_EQ(entry->is_rtx, is_rtx);
RTC_DCHECK_EQ(entry->is_flexfec, is_flexfec);
return entry;
}