1319292 - Enable dormant for dom/media/mediasource/test/test_Eviction_mp4.html

Status: NEW

(Core :: Audio/Video: Playback, defect, P3)

Description

[JW Wang [:jwwang] [:jw_wang]]

Dormant should be invisible to the web. We will figure out how dormant causes test failures.

Comment 1

[JW Wang [:jwwang] [:jw_wang]]

Got this error when dormant is enabled:
120 INFO TEST-UNEXPECTED-FAIL | dom/media/mediasource/test/test_Eviction_mp4.html | Shouldn't throw another time when data can be evicted

This is because:
DormantState::Enter -> MDSM::Reset -> MediaDecoderReaderWrapper::ResetDecode -> MediaFormatReader::ResetDecode -> DecoderData::ResetDemuxer -> MediaSourceTrackDemuxer::Reset -> TrackBuffersManager::Seek -> TrackBuffersManager::ResetEvictionIndex which changes mEvictionIndex.

Comment 2

[Jean-Yves Avenard [:jya]]

hmmm... interesting problem...

either we keep having dormant disable for this test.

Or we always allow an append to succeed after a seek, even if we think the buffer is full, eviction will still happen later in the cycle, just not synchronously.

Comment 3

[JW Wang [:jwwang] [:jw_wang]]

I guess the later has less impact on the user.

Comment 4

[JW Wang [:jwwang] [:jw_wang]]

Hi Jya,
Please take this bug since this is your domain.