Closed
Bug 786690
Opened 12 years ago
Closed 12 years ago
Fix buffered streams when outputting 16bit
Categories
(Core :: Audio/Video, defect)
Core
Audio/Video
Tracking
()
RESOLVED
FIXED
mozilla18
People
(Reporter: mwu, Assigned: mwu)
References
Details
(Keywords: qawanted)
Attachments
(1 file)
609 bytes,
patch
|
kinetik
:
review+
akeybl
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
Assignee | ||
Updated•12 years ago
|
Attachment #656461 -
Flags: review? → review?(kinetik)
Updated•12 years ago
|
Attachment #656461 -
Flags: review?(kinetik) → review+
Assignee | ||
Comment 1•12 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/5071c1a3c394
Target Milestone: --- → mozilla18
Comment 2•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/5071c1a3c394
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Comment on attachment 656461 [details] [diff] [review] Fix buffered streams on 16 bit outputs [Approval Request Comment] Bug caused by (feature/regressing bug #): bug 775319 User impact if declined: garbled/no audio, may crash with some players Testing completed (on m-c, etc.): already in Aurora with no issues; by me on cubeb/alsa with youtube/html5 player using freebsd where float32 doesn't work (bug 780531) Risk to taking this patch (and alternatives if risky): none for float32, garbled/no audio for s16 (e.g. on android)
Attachment #656461 -
Flags: approval-mozilla-beta?
Comment 4•12 years ago
|
||
We'll approve this, but we need to make sure we didn't regress Firefox for Android. Can you provide some example streams for QA to test?
Updated•12 years ago
|
Attachment #656461 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Assignee | ||
Comment 5•12 years ago
|
||
(In reply to Alex Keybl [:akeybl] from comment #4) > We'll approve this, but we need to make sure we didn't regress Firefox for > Android. Can you provide some example streams for QA to test? Firefox for Android doesn't use this code path at all. Otherwise, it would've been discovered and fixed long ago. Firefox for Android uses sydneyaudio while this code path is for cubeb. When this bug is triggered, the audio output is mostly noise. No special example streams are necessary.
Assignee | ||
Comment 6•12 years ago
|
||
(In reply to Jan Beich from comment #3) > Comment on attachment 656461 [details] [diff] [review] > Fix buffered streams on 16 bit outputs > > [Approval Request Comment] > Bug caused by (feature/regressing bug #): bug 775319 > User impact if declined: garbled/no audio, may crash with some players > Testing completed (on m-c, etc.): already in Aurora with no issues; by me on > cubeb/alsa with youtube/html5 player using freebsd where float32 doesn't > work (bug 780531) > Risk to taking this patch (and alternatives if risky): none for float32, > garbled/no audio for s16 (e.g. on android) Are you going to land it or will you need someone to land it for you?
(In reply to Michael Wu [:mwu] from comment #6) > Are you going to land it or will you need someone to land it for you? No, I have zero commit access. Please, land it to 17.0.
Updated•12 years ago
|
QA Contact: aaron.train
Comment 9•12 years ago
|
||
Is there anything specific that we can do here to verify this bug? See comment 4.
Assignee | ||
Comment 10•12 years ago
|
||
(In reply to Virgil Dicu [:virgil] [QA] from comment #9) > Is there anything specific that we can do here to verify this bug? See > comment 4. This bug never existed on any of our tier 1 platforms. It sounds like testing firefox on freebsd might do the trick but I dunno if there's any special configuration required to enable cubeb/alsa.
Comment 11•12 years ago
|
||
I was not able to verify this due to lack of resources and system configuration. Michael, can you please verify if this is fixed?
Comment 12•11 years ago
|
||
mass remove verifyme requests greater than 4 months old
Keywords: verifyme
You need to log in
before you can comment on or make changes to this bug.
Description
•