Closed
Bug 1179498
Opened 9 years ago
Closed 9 years ago
Make all of MediaDecoder run on the main thread
Categories
(Core :: Audio/Video, defect)
Core
Audio/Video
Tracking
()
RESOLVED
FIXED
People
(Reporter: bholley, Unassigned)
References
Details
There's a mostly-trivial long-tail of methods on MediaDecoder that run on other threads. Everything on MediaDecoder needs to run on the main thread.
Here's a quick list of what remains to be fixed:
QueueMetadata
IsExpectingMoreData
IsSameOriginMedia
GetStatistics
ComputePlaybackRate
SetMediaSeekable/IsTransportSeekable/IsMediaSeekable
{Start,Stop}ProgressUpdates
UpdatePlaybackOffset
GetImageContainer/InvalidateWithFlags/Invalidate
IsShutdown
CanPlayThrough
GetCDMProxy
Comment 1•9 years ago
|
||
Here's a quick list of what remains to be fixed:
GetStatistics
ComputePlaybackRate
SetMediaSeekable/IsTransportSeekable/IsMediaSeekable
UpdatePlaybackOffset
GetImageContainer/InvalidateWithFlags/Invalidate
IsShutdown
CanPlayThrough
GetCDMProxy
Comment 2•9 years ago
|
||
What remains after bug 1203877 is fixed:
SetMediaSeekable/IsTransportSeekable/IsMediaSeekable
GetImageContainer/InvalidateWithFlags/Invalidate
IsShutdown
CanPlayThrough
GetCDMProxy
Comment 3•9 years ago
|
||
mVideoFrameContainer assigned in MediaDecoder::Init and will not change again so it is thread safe. We don't have to fix GetImageContainer/InvalidateWithFlags/Invalidate.
What remians:
SetMediaSeekable/IsTransportSeekable/IsMediaSeekable
IsShutdown
CanPlayThrough
GetCDMProxy
Comment 4•9 years ago
|
||
What remians: GetCDMProxy
The last one!
Reporter | ||
Comment 5•9 years ago
|
||
(In reply to JW Wang [:jwwang] from comment #4)
> What remians: GetCDMProxy
> The last one!
\o/
Comment 6•9 years ago
|
||
Resolved as all dependents are fixed.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•