Bug 1261900: [MSE/webm] P6. Don't unnecessarily calculate the next keyframe time. r?kinetik draft
authorJean-Yves Avenard <jyavenard@mozilla.com>
Fri, 22 Apr 2016 16:41:52 +1000
changeset 356743 55d02c8b8c248e9d3e7517ce743141eaa4e94aed
parent 356742 e6a461ea8028284b890d5abaf778a46099af8094
child 356744 33956d40af71e8733df78a58b83822cdc6f95a83
push id16586
push userbmo:jyavenard@mozilla.com
push dateWed, 27 Apr 2016 02:47:40 +0000
reviewerskinetik
bugs1261900
milestone49.0a1
Bug 1261900: [MSE/webm] P6. Don't unnecessarily calculate the next keyframe time. r?kinetik MSE only uses the webm demuxer to demux all samples at once. Attempting to find the next keyframe as such always fail. MozReview-Commit-ID: KKmOpKGde2C
dom/media/webm/WebMDemuxer.cpp
--- a/dom/media/webm/WebMDemuxer.cpp
+++ b/dom/media/webm/WebMDemuxer.cpp
@@ -893,17 +893,17 @@ WebMTrackDemuxer::GetSamples(int32_t aNu
     UpdateSamples(samples->mSamples);
     return SamplesPromise::CreateAndResolve(samples, __func__);
   }
 }
 
 void
 WebMTrackDemuxer::SetNextKeyFrameTime()
 {
-  if (mType != TrackInfo::kVideoTrack) {
+  if (mType != TrackInfo::kVideoTrack || mParent->IsMediaSource()) {
     return;
   }
 
   int64_t frameTime = -1;
 
   mNextKeyframeTime.reset();
 
   MediaRawDataQueue skipSamplesQueue;