Bug 1906088 Comment 5 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

Original comment by
on 
The two assertion failures shortly after 06/20 were in [`MP4SampleIndex::ConvertByteRangesToTimeRanges(mozilla::media::IntervalSet<long long> const&)`](https://hg.mozilla.org/mozilla-central/file/389fe769c55c14205de4ea1c874a63b70abfdbd2/dom/media/mp4/SampleIterator.cpp#l610) [from](https://crash-stats.mozilla.org/report/index/0dcf8efa-0183-4b32-8280-b890f0240624) MP4TrackDemuxer::GetBuffered() and [in]() [`HTMLMediaElement::Buffered(https://crash-stats.mozilla.org/report/index/81f532cf-f7fc-43e2-8e3d-d34630240623)`](https://hg.mozilla.org/releases/mozilla-beta/file/297d4bd38b8663bb54e365cc7f3a0cc7c72fe245/dom/html/HTMLMediaElement.cpp#l6743)

Since changes for bug 1903974, the assertion failures have been in [two](https://hg.mozilla.org/mozilla-central/file/b3b798d8d5231417c7c4c20a70cb7430395a886e/dom/media/mediasource/TrackBuffersManager.cpp#l2541) call [sites](https://hg.mozilla.org/mozilla-central/file/b3b798d8d5231417c7c4c20a70cb7430395a886e/dom/media/mediasource/TrackBuffersManager.cpp#l2554) in `TrackBuffersManager::RemoveFrames()`.

`RemoveFrames()` has a [history of buffered ranges that are inconsistent with `TrackBuffer` sample intervals](https://searchfox.org/mozilla-central/rev/4e0f4c612589c23344f22285520a44a3f45c9e6e/dom/media/mediasource/TrackBuffersManager.cpp#2665-2667,2682-2684), so the assertion failures there might have the same root cause.
Revision 1 by
on 
The two assertion failures shortly after 06/20 were in [`MP4SampleIndex::ConvertByteRangesToTimeRanges(mozilla::media::IntervalSet<long long> const&)`](https://hg.mozilla.org/mozilla-central/file/389fe769c55c14205de4ea1c874a63b70abfdbd2/dom/media/mp4/SampleIterator.cpp#l610) [from](https://crash-stats.mozilla.org/report/index/0dcf8efa-0183-4b32-8280-b890f0240624) MP4TrackDemuxer::GetBuffered() and [in](https://crash-stats.mozilla.org/report/index/81f532cf-f7fc-43e2-8e3d-d34630240623) [`HTMLMediaElement::Buffered()`](https://hg.mozilla.org/releases/mozilla-beta/file/297d4bd38b8663bb54e365cc7f3a0cc7c72fe245/dom/html/HTMLMediaElement.cpp#l6743)

Since changes for bug 1903974, the assertion failures have been in [two](https://hg.mozilla.org/mozilla-central/file/b3b798d8d5231417c7c4c20a70cb7430395a886e/dom/media/mediasource/TrackBuffersManager.cpp#l2541) call [sites](https://hg.mozilla.org/mozilla-central/file/b3b798d8d5231417c7c4c20a70cb7430395a886e/dom/media/mediasource/TrackBuffersManager.cpp#l2554) in `TrackBuffersManager::RemoveFrames()`.

`RemoveFrames()` has a [history of buffered ranges that are inconsistent with `TrackBuffer` sample intervals](https://searchfox.org/mozilla-central/rev/4e0f4c612589c23344f22285520a44a3f45c9e6e/dom/media/mediasource/TrackBuffersManager.cpp#2665-2667,2682-2684), so the assertion failures there might have the same root cause.
Revision 2 by
on 
The two assertion failures shortly after 06/20 were in [`MP4SampleIndex::ConvertByteRangesToTimeRanges(mozilla::media::IntervalSet<long long> const&)`](https://hg.mozilla.org/mozilla-central/file/389fe769c55c14205de4ea1c874a63b70abfdbd2/dom/media/mp4/SampleIterator.cpp#l610) [from](https://crash-stats.mozilla.org/report/index/0dcf8efa-0183-4b32-8280-b890f0240624) MP4TrackDemuxer::GetBuffered() and in [`ToMicrosecondResolution()`](https://hg.mozilla.org/releases/mozilla-beta/file/297d4bd38b8663bb54e365cc7f3a0cc7c72fe245/dom/media/TimeUnits.h#l301) [from](https://crash-stats.mozilla.org/report/index/81f532cf-f7fc-43e2-8e3d-d34630240623) [`HTMLMediaElement::Buffered()`](https://hg.mozilla.org/releases/mozilla-beta/file/297d4bd38b8663bb54e365cc7f3a0cc7c72fe245/dom/html/HTMLMediaElement.cpp#l6743)

Since changes for bug 1903974, the assertion failures have been in [two](https://hg.mozilla.org/mozilla-central/file/b3b798d8d5231417c7c4c20a70cb7430395a886e/dom/media/mediasource/TrackBuffersManager.cpp#l2541) call [sites](https://hg.mozilla.org/mozilla-central/file/b3b798d8d5231417c7c4c20a70cb7430395a886e/dom/media/mediasource/TrackBuffersManager.cpp#l2554) in `TrackBuffersManager::RemoveFrames()`.

`RemoveFrames()` has a [history of buffered ranges that are inconsistent with `TrackBuffer` sample intervals](https://searchfox.org/mozilla-central/rev/4e0f4c612589c23344f22285520a44a3f45c9e6e/dom/media/mediasource/TrackBuffersManager.cpp#2665-2667,2682-2684), so the assertion failures there might have the same root cause.
Revision 3 by
on 
The two assertion failures shortly after 06/20 were in [`MP4SampleIndex::ConvertByteRangesToTimeRanges(mozilla::media::IntervalSet<long long> const&)`](https://hg.mozilla.org/mozilla-central/file/389fe769c55c14205de4ea1c874a63b70abfdbd2/dom/media/mp4/SampleIterator.cpp#l610) [from](https://crash-stats.mozilla.org/report/index/0dcf8efa-0183-4b32-8280-b890f0240624) MP4TrackDemuxer::GetBuffered() and in [`ToMicrosecondResolution()`](https://hg.mozilla.org/releases/mozilla-beta/file/297d4bd38b8663bb54e365cc7f3a0cc7c72fe245/dom/media/TimeUnits.h#l301) [from](https://crash-stats.mozilla.org/report/index/81f532cf-f7fc-43e2-8e3d-d34630240623) [`HTMLMediaElement::Buffered()`](https://hg.mozilla.org/releases/mozilla-beta/file/297d4bd38b8663bb54e365cc7f3a0cc7c72fe245/dom/html/HTMLMediaElement.cpp#l6743).

Since changes for bug 1903974, the assertion failures have been in [two](https://hg.mozilla.org/mozilla-central/file/b3b798d8d5231417c7c4c20a70cb7430395a886e/dom/media/mediasource/TrackBuffersManager.cpp#l2541) call [sites](https://hg.mozilla.org/mozilla-central/file/b3b798d8d5231417c7c4c20a70cb7430395a886e/dom/media/mediasource/TrackBuffersManager.cpp#l2554) in `TrackBuffersManager::RemoveFrames()`.

`RemoveFrames()` has a [history of buffered ranges that are inconsistent with `TrackBuffer` sample intervals](https://searchfox.org/mozilla-central/rev/4e0f4c612589c23344f22285520a44a3f45c9e6e/dom/media/mediasource/TrackBuffersManager.cpp#2665-2667,2682-2684), so the assertion failures there might have the same root cause.
Revision 4 by
on 
The two assertion failures shortly after 06/20 were in [`MP4SampleIndex::ConvertByteRangesToTimeRanges(mozilla::media::IntervalSet<long long> const&)`](https://hg.mozilla.org/mozilla-central/file/389fe769c55c14205de4ea1c874a63b70abfdbd2/dom/media/mp4/SampleIterator.cpp#l610) [from](https://crash-stats.mozilla.org/report/index/0dcf8efa-0183-4b32-8280-b890f0240624) `MP4TrackDemuxer::GetBuffered()` and in [`ToMicrosecondResolution()`](https://hg.mozilla.org/releases/mozilla-beta/file/297d4bd38b8663bb54e365cc7f3a0cc7c72fe245/dom/media/TimeUnits.h#l301) [from](https://crash-stats.mozilla.org/report/index/81f532cf-f7fc-43e2-8e3d-d34630240623) [`HTMLMediaElement::Buffered()`](https://hg.mozilla.org/releases/mozilla-beta/file/297d4bd38b8663bb54e365cc7f3a0cc7c72fe245/dom/html/HTMLMediaElement.cpp#l6743).

Since changes for bug 1903974, the assertion failures have been in [two](https://hg.mozilla.org/mozilla-central/file/b3b798d8d5231417c7c4c20a70cb7430395a886e/dom/media/mediasource/TrackBuffersManager.cpp#l2541) call [sites](https://hg.mozilla.org/mozilla-central/file/b3b798d8d5231417c7c4c20a70cb7430395a886e/dom/media/mediasource/TrackBuffersManager.cpp#l2554) in `TrackBuffersManager::RemoveFrames()`.

`RemoveFrames()` has a [history of buffered ranges that are inconsistent with `TrackBuffer` sample intervals](https://searchfox.org/mozilla-central/rev/4e0f4c612589c23344f22285520a44a3f45c9e6e/dom/media/mediasource/TrackBuffersManager.cpp#2665-2667,2682-2684), so the assertion failures there might have the same root cause.

Back to Bug 1906088 Comment 5