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

Unified Diff: webrtc/modules/congestion_controller/probe_bitrate_estimator_unittest.cc

Issue 2239143002: ProbingEstimator: Erase history based on time threshold (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Created 4 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/congestion_controller/probe_bitrate_estimator_unittest.cc
diff --git a/webrtc/modules/congestion_controller/probe_bitrate_estimator_unittest.cc b/webrtc/modules/congestion_controller/probe_bitrate_estimator_unittest.cc
index e3bbd5a22793e2d8a73c0a277ae02f9fc036acb9..2d82d2ba36c8cf2b265dbfc1e2a1a9a91e091dde 100644
--- a/webrtc/modules/congestion_controller/probe_bitrate_estimator_unittest.cc
+++ b/webrtc/modules/congestion_controller/probe_bitrate_estimator_unittest.cc
@@ -29,9 +29,9 @@ class TestProbeBitrateEstimator : public ::testing::Test {
int64_t arrival_time_ms) {
PacketInfo info(arrival_time_ms, send_time_ms, 0, size_bytes,
probe_cluster_id);
- ProbingResult res = probe_bitrate_estimator_.PacketFeedback(info);
- if (res.valid())
- results_.emplace_back(res.bps, res.timestamp);
+ int bps = probe_bitrate_estimator_.HandleProbeAndEstimateBitrate(info, 1);
+ if (bps > 0)
+ results_.emplace_back(bps, info.arrival_time_ms);
}
void CheckResult(size_t index, int bps, int max_diff, int64_t timestamp) {

Powered by Google App Engine
This is Rietveld 408576698