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

Unified Diff: webrtc/modules/audio_coding/codecs/audio_encoder.h

Issue 1184313002: Add AudioEncoder::GetTargetBitrate (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Getting rid of erroneous code Created 5 years, 6 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/codecs/audio_encoder.h
diff --git a/webrtc/modules/audio_coding/codecs/audio_encoder.h b/webrtc/modules/audio_coding/codecs/audio_encoder.h
index 9025834dfa78704d16f00aed0b5bfe091eda86d7..35fbfad5f7d185b53e6a7e9b30380ad8e3e94bce 100644
--- a/webrtc/modules/audio_coding/codecs/audio_encoder.h
+++ b/webrtc/modules/audio_coding/codecs/audio_encoder.h
@@ -97,8 +97,9 @@ class AudioEncoder {
virtual int Max10MsFramesInAPacket() const = 0;
// Changes the target bitrate. The implementation is free to alter this value,
- // e.g., if the desired value is outside the valid range.
- virtual void SetTargetBitrate(int bits_per_second) {}
+ // e.g., if the desired value is outside the valid range. Returns the new
+ // target bitrate (which may or may not be equal to the input parameter).
+ virtual int SetTargetBitrate(int bits_per_second) = 0;
// Tells the implementation what the projected packet loss rate is. The rate
// is in the range [0.0, 1.0]. This rate is typically used to adjust channel

Powered by Google App Engine
This is Rietveld 408576698