d77ea08468
Eliminate initial notifies
...
Not really needed or helpful for our use.
2018-07-25 07:05:51 +08:00
a5b5468889
Use state-watching machinery for UpdateNextFrameStatus
2018-07-25 07:05:45 +08:00
b415ed83c6
Be more disciplined about where we call UpdateNextFrameStatus
...
This value depends on mState and the audio/video queue states. Given that, we can just stick these calls in a few choke points.
2018-07-25 07:05:43 +08:00
41c67a91d8
Mirror mPlayState and mNextState to the state machine task queue and eliminate cross-thread access
2018-07-25 07:05:41 +08:00
6fa240db4e
Re-land "Dispatch AudioSink notifications asynchronously"
...
Turns out that this is needed after all.
2018-07-25 07:04:46 +08:00
54c849b63d
Revert "Re-land "Dispatch AudioSink notifications asynchronously" + fixes"
2018-07-25 07:04:08 +08:00
1a2fea0658
Re-land "Dispatch AudioSink notifications asynchronously"
...
This is needed by some new code that I am working on.
2018-07-25 07:03:56 +08:00
df074df1ee
Use state mirroring for NextFrameStatus
2018-07-25 07:03:42 +08:00
422ce43cbb
Invoke SeekCompleted synchronously
...
The current behavior is a holdover from when this some of this stuff was happening on the decode thread. It's problematic because nothing guarantees that we don't call CheckIfSeekComplete twice, and queue up two SeekCompleted events.
2018-07-25 07:03:28 +08:00
8b8219a3c1
Make it impossible for the MDSM frame calculation to overflow
2018-07-25 07:02:05 +08:00
dc0c3e6be3
Remove a couple more instances of manually passing TaskDispatchers
...
Followup to 8446234.
2018-07-25 06:58:15 +08:00
b11500bddd
Stop manually passing TaskDispatchers everywhere
2018-07-25 06:57:56 +08:00
dfc7504069
Remove MaybeTailDispatch
...
This is now the default behavior.
2018-07-25 06:57:53 +08:00
87a4cf6376
Use TailDispatch for the MDSM's task queue
2018-07-25 06:55:39 +08:00
7dd99b6694
Stop doing sync dispatch of PlaybackEnded
2018-07-25 06:54:58 +08:00
ba4e0eb433
Add additional assertions to the MDSM
2018-07-25 06:54:28 +08:00
135695cd07
Add missing ,
2018-07-25 06:53:14 +08:00
e1caa768c4
Handle playback ended in a consistent way for both file and stream case
2018-07-25 06:53:09 +08:00
a068fb2b80
Rework and simplify stream clock calculation in MediaDecoderStateMachine
2018-07-25 06:53:05 +08:00
b6cdd50380
Remove wrapper-style AbstractThreads for MediaTaskQueue and have callers pass an AbstractThread directly to promise APIs
2018-07-25 06:52:28 +08:00
2d35bde18f
Rename MDSM::OnDecodeThread to MDSM::OnDecodeTaskQueue
2018-07-25 06:44:53 +08:00
19e0f7aa42
Rename OnStateMachineThread-like functions to reflect the fact that it's a task queue
2018-07-25 06:44:51 +08:00
17e6d71294
Revert "Dispatch AudioSink notifications asynchronously"
2018-07-25 06:36:53 +08:00
fe42099872
Don't unnecessarily call GetBuffered() in MDSM
2018-07-25 01:28:12 +08:00
b20b8dadab
Replace dom::TimeRanges with TimeIntervals object
2018-07-25 01:27:34 +08:00
92235ac152
Rename and make AsyncReadMetadata virtual
2018-07-25 01:24:24 +08:00
7e5b4386f2
Combine MediaInfo and mp4_demuxer::TrackConfig
2018-07-25 01:18:31 +08:00
d5e2cf7abc
Dispatch AudioSink notifications asynchronously
2018-07-25 01:05:51 +08:00
7c9a6dbbb9
Assert state machine thread for SetState
...
We assert in CheckInDecodeComplete as well for good measure, since that's the last caller of SetState that doesn't already assert it.
2018-07-25 01:05:48 +08:00
5155516c96
Make MDSM::StartBuffering happen on the state machine thread
2018-07-25 01:05:46 +08:00
90904cb897
Make MDSM::Shutdown happen on the state machine thread
2018-07-25 01:05:42 +08:00
4979a99ce9
Add missing string to decoder state list
2018-07-25 01:05:16 +08:00
48e4be08d6
Create one unified thread pool for media code and run the MDSM task queues on it
...
This allows for parallel MDSM execution.
2018-07-25 01:05:04 +08:00
2fc84fbc8e
Fix build bustage: Add missing }
2018-07-25 01:04:02 +08:00
de7d6e349b
Unify FlushDecoding, ResetDecode, and ResetPlayback into a single Reset() method
2018-07-25 01:04:00 +08:00
7c704f00af
Remove the AwaitIdle call in FlushDecoding
...
There are two callers of FlushDecoding - One is shutdown (where we already use promises to wait for the queue to go idle via BeginShutdown), and the other is dormant.
2018-07-25 01:03:58 +08:00
07ffc3fc1e
Deliver NotifyWaitingForResourcesStatusChanged asynchronously on the state machine task queue
...
The previous setup is wacky, and can cause the notification to reach the state machine before the promise rejection, which causes us to stall intermittently. We also take the opportunity to be a bit less trigger happy when we fire it in MediaSourceReader.cpp.
2018-07-25 01:03:45 +08:00
74eb7093f6
Use promises for metadata decoding
2018-07-25 01:03:43 +08:00
3ef89dda87
Fire loadedmetadata before encrypted event
2018-07-25 01:03:42 +08:00
b944f99a23
Stop synchronously dispatching MediaDecoder::DecodeError from MDSM::DecodeError
...
MediaDecoder::DecodeError invokes MediaDecoder::Shutdown, which shuts down the state machine. Given the sync dispatch, this means that this is basically already what's happening.
2018-07-25 01:03:40 +08:00
770ada6bb1
Run MDSM disposal off the MediaPromise returned by initiating shutdown on the task queue
2018-07-25 01:03:36 +08:00
b265ff9e74
Switch the MDSM to a task queue
2018-07-25 01:03:34 +08:00
2d0902db6e
Make the MediaDecoder-to-MediaDecoderStateMachine interface Promise-based
2018-07-25 01:03:21 +08:00
32f141833f
Move the RenderVideoFrame call to the end of SeekCompleted
...
This means that we can get rid of the code to recheck state after dropping the monitor. We'll remove the other monitor drop from this method in a future commit.
2018-07-25 01:03:10 +08:00
8acc4e725d
Move some work from MDSM::Seek to MDSM::InitiateSeek
...
This has two implications:
1. We no longer need to pipe mQueuedSeekTarget through MDSM::Seek to get the appropriate clamping.
2. MDSM::Seek doesn't _need_ to be called on the main thread anymore.
2018-07-25 01:03:08 +08:00
3738ea58fc
Cleanup some comments and assertions that are no longer needed
2018-07-25 01:03:06 +08:00
1d38e973bb
Streamline seek initiation logic and abolish manual seek cancels and retries
...
The model we're moving towards is one where the MDSM can just disconnect all of its promises, send a ResetDecode down the pipe, and start doing something unrelated.
2018-07-25 01:03:03 +08:00
a4dcd6ac96
Stop doing sync dispatch for SeekingStarted
2018-07-25 01:03:01 +08:00
2ef67ac717
Remove silly special case for seeking to exactly the current time
2018-07-25 01:02:59 +08:00
b7524cc78e
Route queued seeks back through MDSM::Seek and eliminate MDSM::StartSeek
2018-07-25 01:02:56 +08:00
trav90
trav90
Moonchild