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

Unified Diff: webrtc/modules/audio_coding/audio_network_adaptor/audio_network_adaptor_impl.h

Issue 2661043003: Allow ANA to receive RPLR (recoverable packet loss rate) indications (Closed)
Patch Set: Rebased Created 3 years, 9 months 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/modules/audio_coding/audio_network_adaptor/audio_network_adaptor_impl.h
diff --git a/webrtc/modules/audio_coding/audio_network_adaptor/audio_network_adaptor_impl.h b/webrtc/modules/audio_coding/audio_network_adaptor/audio_network_adaptor_impl.h
index 4bd64ee327d0754c6c888d70c2e9e4aee9422027..f4e842b263417feb1463d2bb3698d6703649fc80 100644
--- a/webrtc/modules/audio_coding/audio_network_adaptor/audio_network_adaptor_impl.h
+++ b/webrtc/modules/audio_coding/audio_network_adaptor/audio_network_adaptor_impl.h
@@ -46,6 +46,13 @@ class AudioNetworkAdaptorImpl final : public AudioNetworkAdaptor {
void SetUplinkPacketLossFraction(
const rtc::Optional<float>& uplink_packet_loss_fraction) override;
+ // TODO(elad.alon): Before landing, discuss s/fraction/rate (everywhere)
+ // with Minyue. If we do, it would both be clearer, as well as make this short
+ // enough that we don't need to spill the line over like that.
+ void SetUplinkRecoverablePacketLossFraction(
+ const rtc::Optional<float>& uplink_recoverable_packet_loss_fraction)
+ override;
elad.alon_webrtc.org 2017/03/16 19:01:37 Minyue, what do you think of s/fraction/rate here?
+
void SetRtt(int rtt_ms) override;
void SetTargetAudioBitrate(int target_audio_bitrate_bps) override;

Powered by Google App Engine
This is Rietveld 408576698