Bug 1259916: [MSE] P2. Bump audio source buffer eviction threshold to 30MB. r=gerald
authorJean-Yves Avenard <jyavenard@mozilla.com>
Mon, 28 Mar 2016 00:48:57 +1100
changeset 290691 732f589f694e3a5af43dacf868523bde606f5e62
parent 290690 d864ffed536fb108b28a3a0e1312f4d612ca8942
child 290692 bcd0f253bb32a6d1db5235dd03ab5996b6e0efbf
push id19656
push usergwagner@mozilla.com
push dateMon, 04 Apr 2016 13:43:23 +0000
treeherderb2g-inbound@e99061fde28a [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
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()
 {