From 358e71747a4707f9429b8565a05482c1f68d0ed3 Mon Sep 17 00:00:00 2001 From: Chong Zhang Date: Mon, 9 Mar 2015 09:08:15 -0700 Subject: do not do bandwidth estimation if total data downloaded is too small bug: 19656539 Change-Id: I0ab9baad31e8953224a37d5aaf816e44780191c0 --- media/libstagefright/HTTPBase.cpp | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) (limited to 'media') diff --git a/media/libstagefright/HTTPBase.cpp b/media/libstagefright/HTTPBase.cpp index 0c2ff15..77a652a 100644 --- a/media/libstagefright/HTTPBase.cpp +++ b/media/libstagefright/HTTPBase.cpp @@ -75,7 +75,11 @@ void HTTPBase::addBandwidthMeasurement( bool HTTPBase::estimateBandwidth(int32_t *bandwidth_bps) { Mutex::Autolock autoLock(mLock); - if (mNumBandwidthHistoryItems < 2) { + // Do not do bandwidth estimation if we don't have enough samples, or + // total bytes download are too small (<64K). + // Bandwidth estimation from these samples can often shoot up and cause + // unwanted bw adaption behaviors. + if (mNumBandwidthHistoryItems < 2 || mTotalTransferBytes < 65536) { return false; } -- cgit v1.1