Closed Bug 1204430 Opened 4 years ago Closed 4 years ago

Make MediaDecoder::IsMediaSeekable run on the main thread.

Categories

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

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla44
Tracking Status
firefox44 --- fixed

People

(Reporter: jwwang, Assigned: jwwang)

References

Details

Attachments

(2 files)

No description provided.
Assignee: nobody → jwwang
Blocks: 1179498
Depends on: 1204434
Bug 1204430. Part 1 - dispatch AbstractMediaDecoder::SetMediaSeekable() to the main thread.
Attachment #8663482 - Flags: review?(kinetik)
Bug 1204430. Part 2 - mirror MediaDecoder::mMediaSeekable.
Attachment #8663483 - Flags: review?(kinetik)
Comment on attachment 8663482 [details]
MozReview Request: Bug 1204430. Part 1 - dispatch AbstractMediaDecoder::SetMediaSeekable() to the main thread.

https://reviewboard.mozilla.org/r/19819/#review17799

I assume this doesn't use a mirror because it's only set once?
Attachment #8663482 - Flags: review?(kinetik) → review+
Comment on attachment 8663483 [details]
MozReview Request: Bug 1204430. Part 2 - mirror MediaDecoder::mMediaSeekable.

https://reviewboard.mozilla.org/r/19821/#review17801

Disregard previous question. :-)
Attachment #8663483 - Flags: review?(kinetik) → review+
Thanks for the review!
https://hg.mozilla.org/mozilla-central/rev/6dc0c2447593
https://hg.mozilla.org/mozilla-central/rev/1f9594433a39
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla44
You need to log in before you can comment on or make changes to this bug.