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

Unified Diff: webrtc/call/call.cc

Issue 2437503004: Set actual transport overhead in rtp_rtcp (Closed)
Patch Set: Rename SignalTransportOverheadChanged to UpdateTransportOverhead. 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
« no previous file with comments | « webrtc/call.h ('k') | webrtc/media/base/fakemediaengine.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/call/call.cc
diff --git a/webrtc/call/call.cc b/webrtc/call/call.cc
index d7272170841c665883fff1e4ceae0adb1e12d66b..ebc1aeb866237b9c8c16abef5a5499419c228a62 100644
--- a/webrtc/call/call.cc
+++ b/webrtc/call/call.cc
@@ -111,6 +111,9 @@ class Call : public webrtc::Call,
void SignalChannelNetworkState(MediaType media, NetworkState state) override;
+ void OnTransportOverheadChanged(MediaType media,
+ int transport_overhead_per_packet) override;
+
void OnNetworkRouteChanged(const std::string& transport_name,
const rtc::NetworkRoute& network_route) override;
@@ -736,6 +739,30 @@ void Call::SignalChannelNetworkState(MediaType media, NetworkState state) {
}
}
+void Call::OnTransportOverheadChanged(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) {
« no previous file with comments | « webrtc/call.h ('k') | webrtc/media/base/fakemediaengine.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698