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

Unified Diff: webrtc/modules/rtp_rtcp/include/rtp_rtcp_defines.h

Issue 2061423003: Refactor NACK bitrate allocation (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Moved rate limiter and addressed comments Created 4 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/rtp_rtcp/include/rtp_rtcp_defines.h
diff --git a/webrtc/modules/rtp_rtcp/include/rtp_rtcp_defines.h b/webrtc/modules/rtp_rtcp/include/rtp_rtcp_defines.h
index 05320f72e1b5a5272bf7c35a2330ce7abe1d6f65..cbed14351089576eb289fce44c95867979bcfbf4 100644
--- a/webrtc/modules/rtp_rtcp/include/rtp_rtcp_defines.h
+++ b/webrtc/modules/rtp_rtcp/include/rtp_rtcp_defines.h
@@ -397,5 +397,15 @@ class TransportSequenceNumberAllocator {
virtual uint16_t AllocateSequenceNumber() = 0;
};
+class NackRateLimiter {
danilchap 2016/06/28 14:26:18 Probably Retransmission instead of Nack. Since it
sprang_webrtc 2016/07/04 09:33:03 Done.
+ public:
+ NackRateLimiter() {}
+ virtual ~NackRateLimiter() {}
+
+ // Try to use rate to send bytes. Returns true on success and if so updates
+ // current rate.
danilchap 2016/06/28 14:26:18 bytes -> |bytes|. May be rename it to |packet_size
sprang_webrtc 2016/07/04 09:33:03 Done.
+ virtual bool TryUseRate(size_t bytes) = 0;
+};
+
} // namespace webrtc
#endif // WEBRTC_MODULES_RTP_RTCP_INCLUDE_RTP_RTCP_DEFINES_H_

Powered by Google App Engine
This is Rietveld 408576698