Bug 1259916: [MSE] P2. Bump audio source buffer eviction threshold to 30MB. r?gerald draft
authorJean-Yves Avenard <jyavenard@mozilla.com>
Mon, 28 Mar 2016 00:48:57 +1100
changeset 345014 354e7cb57c14ef8e2c5ba06d0f07b17915c4e341
parent 345013 52e23ec8be9d9edacef87fe971e79ad0092bdee1
child 345015 8cfdf120b5373e729914d242217da85c2076272e
push id13994
push userbmo:jyavenard@mozilla.com
push dateSun, 27 Mar 2016 14:00:13 +0000
reviewersgerald
bugs1259916
milestone48.0a1
Bug 1259916: [MSE] P2. Bump audio source buffer eviction threshold to 30MB. r?gerald 20MB appears to work, but just to be safe until we get confirmation from YouTube on what is a safe value to use. MozReview-Commit-ID: BnGaVLeCRJ5
dom/media/mediasource/TrackBuffersManager.cpp
--- a/dom/media/mediasource/TrackBuffersManager.cpp
+++ b/dom/media/mediasource/TrackBuffersManager.cpp
@@ -98,17 +98,17 @@ TrackBuffersManager::TrackBuffersManager
   , mProcessedInput(0)
   , mTaskQueue(aParentDecoder->GetDemuxer()->GetTaskQueue())
   , mParentDecoder(new nsMainThreadPtrHolder<MediaSourceDecoder>(aParentDecoder, false /* strict */))
   , mEnded(false)
   , mDetached(false)
   , mVideoEvictionThreshold(Preferences::GetUint("media.mediasource.eviction_threshold.video",
                                                  100 * 1024 * 1024))
   , mAudioEvictionThreshold(Preferences::GetUint("media.mediasource.eviction_threshold.audio",
-                                                 15 * 1024 * 1024))
+                                                 30 * 1024 * 1024))
   , mEvictionOccurred(false)
   , mMonitor("TrackBuffersManager")
 {
   MOZ_ASSERT(NS_IsMainThread(), "Must be instanciated on the main thread");
 }
 
 TrackBuffersManager::~TrackBuffersManager()
 {