Closed
Bug 1091976
Opened 11 years ago
Closed 11 years ago
[MSE] MP4Reader::NotifyDataArrived() should not dispatch to a separate thread
Categories
(Core :: Audio/Video, defect)
Tracking
()
RESOLVED
FIXED
mozilla36
People
(Reporter: ajones, Assigned: ajones)
References
Details
Attachments
(1 file, 2 obsolete files)
|
7.21 KB,
patch
|
eflores
:
review+
|
Details | Diff | Splinter Review |
MP4Reader::NotifyDataArrived() currently dispatches UpdateIndex() to a separate thread. This causes updates to MP4Reader::GetBuffered() to be delayed which in turn breaks MSE buffer switching.
|
Assignee | |
Comment 1•11 years ago
|
||
|
|
Assignee | |
Updated•11 years ago
|
Assignee: nobody → ajones
Status: NEW → ASSIGNED
|
|
Assignee | |
Comment 2•11 years ago
|
||
|
|
Assignee | |
Updated•11 years ago
|
Attachment #8514737 -
Attachment is obsolete: true
|
|
Assignee | |
Comment 3•11 years ago
|
||
|
|
Assignee | |
Updated•11 years ago
|
Attachment #8514739 -
Attachment is obsolete: true
|
|
Assignee | |
Updated•11 years ago
|
Attachment #8514744 -
Flags: review?(edwin)
Comment on attachment 8514744 [details] [diff] [review]
Process MP4Reader::NotifyDataArrived() immediately
Review of attachment 8514744 [details] [diff] [review]:
-----------------------------------------------------------------
This, but in reverse: http://mozillamemes.tumblr.com/post/45116480402/could-we-could-we-put-them-in-the-cloud
What could possibly go wrong?
Attachment #8514744 -
Flags: review?(edwin) → review+
|
|
Assignee | |
Comment 5•11 years ago
|
||
|
||
Comment 6•11 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla36
You need to log in
before you can comment on or make changes to this bug.
Description
•