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

Unified Diff: webrtc/modules/pacing/alr_detector.h

Issue 2503643003: More reliable ALR detection (Closed)
Patch Set: const 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
« no previous file with comments | « no previous file | webrtc/modules/pacing/alr_detector.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/modules/pacing/alr_detector.h
diff --git a/webrtc/modules/pacing/alr_detector.h b/webrtc/modules/pacing/alr_detector.h
index 6250cc90530ae781f201823d133a5701909d5d3d..31b1644d3020fc90a47b920378e91e1f5f9a4dc4 100644
--- a/webrtc/modules/pacing/alr_detector.h
+++ b/webrtc/modules/pacing/alr_detector.h
@@ -11,6 +11,7 @@
#ifndef WEBRTC_MODULES_PACING_ALR_DETECTOR_H_
#define WEBRTC_MODULES_PACING_ALR_DETECTOR_H_
+#include "webrtc/base/rate_statistics.h"
#include "webrtc/common_types.h"
#include "webrtc/modules/pacing/paced_sender.h"
#include "webrtc/typedefs.h"
@@ -28,19 +29,19 @@ class AlrDetector {
public:
AlrDetector();
~AlrDetector();
- void OnBytesSent(size_t bytes_sent, int64_t elapsed_time_ms);
+
+ void OnBytesSent(size_t bytes_sent, int64_t now_ms);
+
// Set current estimated bandwidth.
void SetEstimatedBitrate(int bitrate_bps);
+
// Returns true if currently in application-limited region.
- bool InApplicationLimitedRegion();
+ bool InApplicationLimitedRegion() const;
private:
- size_t measurement_interval_bytes_sent_;
- int64_t measurement_interval_elapsed_time_ms_;
- int estimated_bitrate_bps_;
- // Number of consecutive periods over which we observe traffic is application
- // limited.
- int application_limited_count_;
+ RateStatistics rate_;
+ int estimated_bitrate_bps_ = 0;
+ bool application_limited_ = false;
};
} // namespace webrtc
« no previous file with comments | « no previous file | webrtc/modules/pacing/alr_detector.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698