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

Side by Side Diff: webrtc/modules/remote_bitrate_estimator/test/estimators/remb.cc

Issue 1886783002: Fix bug when the BWE times out due to no incoming packets. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Add dchecks. Created 4 years, 8 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 unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright (c) 2015 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2015 The WebRTC project authors. All Rights Reserved.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license 4 * Use of this source code is governed by a BSD-style license
5 * that can be found in the LICENSE file in the root of the source 5 * that can be found in the LICENSE file in the root of the source
6 * tree. An additional intellectual property rights grant can be found 6 * tree. An additional intellectual property rights grant can be found
7 * in the file PATENTS. All contributing project authors may 7 * in the file PATENTS. All contributing project authors may
8 * be found in the AUTHORS file in the root of the source tree. 8 * be found in the AUTHORS file in the root of the source tree.
9 */ 9 */
10 10
(...skipping 51 matching lines...) Expand 10 before | Expand all | Expand 10 after
62 } 62 }
63 63
64 RembReceiver::RembReceiver(int flow_id, bool plot) 64 RembReceiver::RembReceiver(int flow_id, bool plot)
65 : BweReceiver(flow_id), 65 : BweReceiver(flow_id),
66 estimate_log_prefix_(), 66 estimate_log_prefix_(),
67 plot_estimate_(plot), 67 plot_estimate_(plot),
68 clock_(0), 68 clock_(0),
69 recv_stats_(ReceiveStatistics::Create(&clock_)), 69 recv_stats_(ReceiveStatistics::Create(&clock_)),
70 latest_estimate_bps_(-1), 70 latest_estimate_bps_(-1),
71 last_feedback_ms_(-1), 71 last_feedback_ms_(-1),
72 estimator_(new RemoteBitrateEstimatorAbsSendTime(this, &clock_)) { 72 estimator_(new RemoteBitrateEstimatorAbsSendTime(this)) {
73 std::stringstream ss; 73 std::stringstream ss;
74 ss << "Estimate_" << flow_id_ << "#1"; 74 ss << "Estimate_" << flow_id_ << "#1";
75 estimate_log_prefix_ = ss.str(); 75 estimate_log_prefix_ = ss.str();
76 // Default RTT in RemoteRateControl is 200 ms ; 50 ms is more realistic. 76 // Default RTT in RemoteRateControl is 200 ms ; 50 ms is more realistic.
77 estimator_->OnRttUpdate(50, 50); 77 estimator_->OnRttUpdate(50, 50);
78 estimator_->SetMinBitrate(kRemoteBitrateEstimatorMinBitrateBps); 78 estimator_->SetMinBitrate(kRemoteBitrateEstimatorMinBitrateBps);
79 } 79 }
80 80
81 RembReceiver::~RembReceiver() { 81 RembReceiver::~RembReceiver() {
82 } 82 }
(...skipping 70 matching lines...) Expand 10 before | Expand all | Expand 10 after
153 } 153 }
154 latest_estimate_bps_ = bps; 154 latest_estimate_bps_ = bps;
155 } 155 }
156 *estimate_bps = latest_estimate_bps_; 156 *estimate_bps = latest_estimate_bps_;
157 return true; 157 return true;
158 } 158 }
159 159
160 } // namespace bwe 160 } // namespace bwe
161 } // namespace testing 161 } // namespace testing
162 } // namespace webrtc 162 } // namespace webrtc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698