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

Unified Diff: webrtc/modules/congestion_controller/probe_controller.h

Issue 2504023002: Implement periodic bandwidth probing in application-limited region. (Closed)
Patch Set: Address feedback 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
Index: webrtc/modules/congestion_controller/probe_controller.h
diff --git a/webrtc/modules/congestion_controller/probe_controller.h b/webrtc/modules/congestion_controller/probe_controller.h
index e60a007b9d94402253d1982b27f3649f79bb6f32..bcdc2fdf0353a5bba736227454409a804c1949c6 100644
--- a/webrtc/modules/congestion_controller/probe_controller.h
+++ b/webrtc/modules/congestion_controller/probe_controller.h
@@ -13,6 +13,7 @@
#include <initializer_list>
+#include "webrtc/base/optional.h"
stefan-webrtc 2016/11/28 13:27:13 Not used
Sergey Ulanov 2016/11/28 20:39:00 Done.
#include "webrtc/base/criticalsection.h"
#include "webrtc/common_types.h"
#include "webrtc/modules/pacing/paced_sender.h"
@@ -36,6 +37,9 @@ class ProbeController {
void SetEstimatedBitrate(int bitrate_bps);
+ void EnablePeriodicAlrProbing(bool enable);
+ void Process();
+
private:
enum class State {
// Initial state where no probing has been triggered yet.
@@ -47,7 +51,8 @@ class ProbeController {
};
void InitiateExponentialProbing() EXCLUSIVE_LOCKS_REQUIRED(critsect_);
- void InitiateProbing(std::initializer_list<int> bitrates_to_probe,
+ void InitiateProbing(int64_t now_ms,
+ std::initializer_list<int> bitrates_to_probe,
int min_bitrate_to_probe_further_bps)
EXCLUSIVE_LOCKS_REQUIRED(critsect_);
@@ -62,6 +67,7 @@ class ProbeController {
int start_bitrate_bps_ GUARDED_BY(critsect_);
int max_bitrate_bps_ GUARDED_BY(critsect_);
int64_t last_alr_probing_time_ GUARDED_BY(critsect_);
+ bool enable_periodic_alr_probing_ GUARDED_BY(critsect_);
RTC_DISALLOW_IMPLICIT_CONSTRUCTORS(ProbeController);
};

Powered by Google App Engine
This is Rietveld 408576698