MediaController::IsPlaying() can't reflect its playing state correctly
Categories
(Core :: Audio/Video: Playback, task, P2)
Tracking
()
People
(Reporter: alwu, Assigned: alwu)
References
Details
Attachments
(5 obsolete files)
If we control media controller by its public methods only, such as Play()
, Pause()
and Stop()
, then media controller's IsPlaying()
can reflect the playing state correctly.
However, if we pause controller by Pause()
, but resume media from page directly, then the media controller's mIsPlaying
won't be modified, so IsPlaying()
would return incorrect result.
Assignee | ||
Comment 1•5 years ago
|
||
Introduce ControlledMediaState
to reflect different status of controlled media.
eStarted
and eStopped
are used for indicating controlled media started and stopped respectively, and eResumed
is used when we paused media from media controller before, but resume it from page directly via play()
, then we have to update the media controller playing state.
In addition, create a class MediaControlUtils
to wrap all those helper functions to show they are used for same module.
Assignee | ||
Comment 2•5 years ago
|
||
Assignee | ||
Comment 3•5 years ago
|
||
Although all controllers we created and added to MediaControlService
are all started, but we will support adding non-started controller to MediaControlService
in the future, in order to support using MediaSession
's metadata.
Therefore, add a new method to check if the controller is started.
Updated•5 years ago
|
Assignee | ||
Comment 4•5 years ago
|
||
Assignee | ||
Comment 5•5 years ago
|
||
Controller can control multiple controlled media at the same time, if any of them is playing, then the controller should be playing
as well.
Updated•5 years ago
|
Comment 6•5 years ago
|
||
Comment on attachment 9114673 [details]
Bug 1577367 - part3 : only update media action to content process when we have controlled media.
Revision D56434 was moved to bug 1603878. Setting attachment 9114673 [details] to obsolete.
Assignee | ||
Comment 7•5 years ago
|
||
This issue will be fixed in bug1571493.
Updated•5 years ago
|
Updated•5 years ago
|
Updated•5 years ago
|
Updated•5 years ago
|
Description
•