Closed Bug 1202540 Opened 4 years ago Closed 4 years ago

Remove MDSM::mPlayStartTime

Categories

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

defect

Tracking

()

RESOLVED FIXED
mozilla43
Tracking Status
firefox43 --- fixed

People

(Reporter: jwwang, Assigned: jwwang)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

By adding MediaSink::IsPlay(), MDSM can now always ask MediaSink to know whether playback is paused or not. And then we are able to remove |mPlayStartTime|.
s/MediaSink::IsPlay/MediaSink::IsPlaying/
Blocks: 1199098
Depends on: 1202351
Bug 1202540 - Remove MDSM::mPlayStartTime.
Attachment #8659027 - Flags: review?(kinetik)
Comment on attachment 8659027 [details]
MozReview Request: Bug 1202540 - Remove MDSM::mPlayStartTime.

https://reviewboard.mozilla.org/r/18739/#review16781
Attachment #8659027 - Flags: review?(kinetik) → review+
Thanks for the review!
Assignee: nobody → jwwang
Priority: -- → P2
https://hg.mozilla.org/mozilla-central/rev/e5f7eabe6538
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla43
You need to log in before you can comment on or make changes to this bug.