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

Unified Diff: webrtc/modules/remote_bitrate_estimator/test/packet_sender.h

Issue 2999073002: Tweaked version of BBR for WebRTC. (Closed)
Patch Set: Updated according to comments. Created 3 years, 4 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/remote_bitrate_estimator/test/packet_sender.h
diff --git a/webrtc/modules/remote_bitrate_estimator/test/packet_sender.h b/webrtc/modules/remote_bitrate_estimator/test/packet_sender.h
index 0df61b039a4377f6a77e30f3f7490a303a06b9ed..86ad0e812f70d93fcc89859b8f4858482f1e753c 100644
--- a/webrtc/modules/remote_bitrate_estimator/test/packet_sender.h
+++ b/webrtc/modules/remote_bitrate_estimator/test/packet_sender.h
@@ -81,7 +81,6 @@ class VideoSender : public PacketSender, public BitrateObserver {
void OnNetworkChanged(uint32_t target_bitrate_bps,
uint8_t fraction_lost,
int64_t rtt) override;
-
void Pause() override;
void Resume(int64_t paused_time_ms) override;
@@ -123,12 +122,23 @@ class PacedVideoSender : public VideoSender, public PacedSender::PacketSender {
uint8_t fraction_lost,
int64_t rtt) override;
+ void OnNetworkChanged(uint32_t bitrate_for_encoder_bps,
+ uint32_t bitrate_for_pacer_bps,
+ bool in_probe_rtt,
+ int64_t rtt,
+ uint64_t congestion_window) override;
+ size_t pacer_queue_size_in_bytes() override {
+ return pacer_queue_size_in_bytes_;
+ }
+ void OnBytesAcked(size_t bytes) override;
+
private:
int64_t TimeUntilNextProcess(const std::list<Module*>& modules);
void CallProcess(const std::list<Module*>& modules);
void QueuePackets(Packets* batch, int64_t end_of_batch_time_us);
- PacedSender pacer_;
+ size_t pacer_queue_size_in_bytes_ = 0;
+ std::unique_ptr<Pacer> pacer_;
Packets queue_;
Packets pacer_queue_;
« no previous file with comments | « webrtc/modules/remote_bitrate_estimator/test/packet.h ('k') | webrtc/modules/remote_bitrate_estimator/test/packet_sender.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698