Index: webrtc/modules/pacing/alr_detector.cc |
diff --git a/webrtc/modules/pacing/alr_detector.cc b/webrtc/modules/pacing/alr_detector.cc |
index b31d4482bba37896bb7b0709d7e796d6e8c6eb53..9a917983be20c53169100e7e1001eeeafd8b9997 100644 |
--- a/webrtc/modules/pacing/alr_detector.cc |
+++ b/webrtc/modules/pacing/alr_detector.cc |
@@ -36,6 +36,10 @@ AlrDetector::AlrDetector() |
AlrDetector::~AlrDetector() {} |
void AlrDetector::OnBytesSent(size_t bytes_sent, int64_t now_ms) { |
+ // TODO(nisse): It's unclear what guarantees there are that this |
+ // function isn't called before SetEstimatedBitrate. Document that |
+ // guarantee, if it exists. Or else, remove the DCHECK and tolerate |
+ // that current estimate is zero, e.g., by just returning false. |
RTC_DCHECK(estimated_bitrate_bps_); |
rate_.Update(bytes_sent, now_ms); |