Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(387)

Unified Diff: webrtc/call/call.cc

Issue 2437503004: Set actual transport overhead in rtp_rtcp (Closed)
Patch Set: Response to comments of honghaiz3 Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: webrtc/call/call.cc
diff --git a/webrtc/call/call.cc b/webrtc/call/call.cc
index 489983e7edc1a353dda18eea4c02608e147267f8..4f12620cbd3ce2886f24410d03f7d5f763f4bf88 100644
--- a/webrtc/call/call.cc
+++ b/webrtc/call/call.cc
@@ -102,6 +102,10 @@ class Call : public webrtc::Call,
void SignalChannelNetworkState(MediaType media, NetworkState state) override;
+ void SignalTransportOverheadChange(
+ MediaType media,
+ int transport_overhead_per_packet) override;
+
void OnNetworkRouteChanged(const std::string& transport_name,
const rtc::NetworkRoute& network_route) override;
@@ -666,6 +670,30 @@ void Call::SignalChannelNetworkState(MediaType media, NetworkState state) {
}
}
+void Call::SignalTransportOverheadChange(MediaType media,
+ int transport_overhead_per_packet) {
+ switch (media) {
+ case MediaType::AUDIO: {
+ ReadLockScoped read_lock(*send_crit_);
+ for (auto& kv : audio_send_ssrcs_) {
+ kv.second->SetTransportOverhead(transport_overhead_per_packet);
+ }
+ break;
+ }
+ case MediaType::VIDEO: {
+ ReadLockScoped read_lock(*send_crit_);
+ for (auto& kv : video_send_ssrcs_) {
+ kv.second->SetTransportOverhead(transport_overhead_per_packet);
+ }
+ break;
+ }
+ case MediaType::ANY:
+ case MediaType::DATA:
+ RTC_NOTREACHED();
+ break;
+ }
+}
+
// TODO(honghaiz): Add tests for this method.
void Call::OnNetworkRouteChanged(const std::string& transport_name,
const rtc::NetworkRoute& network_route) {

Powered by Google App Engine
This is Rietveld 408576698