If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

DecodeMetadata function will be interrupted by dormant.

RESOLVED INVALID

Status

()

Core
Audio/Video
RESOLVED INVALID
3 years ago
2 years ago

People

(Reporter: bechen, Assigned: bechen)

Tracking

Trunk
x86_64
Gonk (Firefox OS)
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

3 years ago
Fork from bug 1129376 comment 2. Attachment 8562627 [details] [diff].

It's a timing/threading issue between mainthread and decodethread.

https://dxr.mozilla.org/mozilla-central/source/dom/media/MediaDecoderStateMachine.cpp#2169

Then the document's activity change
https://dxr.mozilla.org/mozilla-central/source/dom/media/MediaDecoderStateMachine.cpp#1456

At DecodeMetadata() function, decodethread release the monitor then call mReader->Readmatadata, at the same time on mainthread, MDSM receive "EnterDormant" and "LeaveDormant" cause the state transits to DECODER_STATE_DECODING_NONE. Then back to decoderthread it fails to enter waiting state. 
https://dxr.mozilla.org/mozilla-central/source/dom/media/MediaDecoderStateMachine.cpp#2174
(Assignee)

Comment 1

3 years ago
Created attachment 8565325 [details] [diff] [review]
bug-1132369.v01.patch
(Assignee)

Updated

3 years ago
Assignee: nobody → bechen
(Assignee)

Updated

2 years ago
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.