Closed Bug 1412204 Opened 2 years ago Closed 2 years ago

Add a member to ChannelMediaResource to indicate Close() has been called

Categories

(Core :: Audio/Video: Playback, enhancement, P3)

enhancement

Tracking

()

RESOLVED FIXED
mozilla58
Tracking Status
firefox58 --- fixed

People

(Reporter: jwwang, Assigned: jwwang)

References

Details

Attachments

(1 file)

So we can check this member without calling mCacheStream.IsClosed() which need to acquire the lock and might block the main thread.
Assignee: nobody → jwwang
Blocks: 1369263
Priority: -- → P3
Attachment #8923726 - Flags: review?(gsquelart)
Comment on attachment 8923726 [details]
Bug 1412204 - add a member to ChannelMediaResource to indicate Close() has been called.

https://reviewboard.mozilla.org/r/194872/#review199924
Attachment #8923726 - Flags: review?(gsquelart) → review+
Pushed by jwwang@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/97154c6aa834
add a member to ChannelMediaResource to indicate Close() has been called. r=gerald
https://hg.mozilla.org/mozilla-central/rev/97154c6aa834
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla58
You need to log in before you can comment on or make changes to this bug.