Closed
Bug 1057612
Opened 10 years ago
Closed 10 years ago
Tell Apple AAC decoder about seeks.
Categories
(Core :: Audio/Video, defect)
Tracking
()
RESOLVED
FIXED
mozilla34
People
(Reporter: rillian, Assigned: rillian)
Details
Attachments
(1 file, 1 obsolete file)
(deleted),
patch
|
cpearce
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Assignee | ||
Comment 1•10 years ago
|
||
According to docs, where supposed to flag input discontinuities to AudioFileStreamParseBytes(). Set a flag on flush and use it to add the appropriate value the next time SubmitSample runs.
In theory this races since ::Flush and ::SubmitSample run out of different TaskQueues, but we've just flushed mTaskQueue when we set it, so the change should be seen correctly.
Assignee: nobody → giles
Attachment #8477718 -
Flags: review?(cpearce)
Comment 2•10 years ago
|
||
Comment on attachment 8477718 [details] [diff] [review]
Signal discontinuities to the AudioFileStreamParseBytes
Review of attachment 8477718 [details] [diff] [review]:
-----------------------------------------------------------------
I think for thread safety, you should actually put a task onto the queue that sets mFlushed to true on the task queue, rather than setting it on the decode task queue thread and hoping there's a memory barrier in between setting it and it next being read on the other task queue thread when it's used. There probably will be a memory barrier, but I think it's best to be careful. You could also make it atomic.
Attachment #8477718 -
Flags: review?(cpearce) → review-
Assignee | ||
Comment 3•10 years ago
|
||
Dispatch a task to update mFlushed to avoid thread safety concerns.
Attachment #8477718 -
Attachment is obsolete: true
Attachment #8477747 -
Flags: review?(cpearce)
Comment 4•10 years ago
|
||
Comment on attachment 8477747 [details] [diff] [review]
Signal discontinuities to the AudioFileStreamParseBytes
Review of attachment 8477747 [details] [diff] [review]:
-----------------------------------------------------------------
Thanks.
Attachment #8477747 -
Flags: review?(cpearce) → review+
Assignee | ||
Comment 5•10 years ago
|
||
Comment 6•10 years ago
|
||
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla34
You need to log in
before you can comment on or make changes to this bug.
Description
•